Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removal of the callout in the in "Summary and Setup" page #161

Closed
wants to merge 1,167 commits into from
Closed

Removal of the callout in the in "Summary and Setup" page #161

wants to merge 1,167 commits into from

Conversation

mimkes
Copy link
Contributor

@mimkes mimkes commented Aug 30, 2023

Closes ##154

I have removed the callout as suggested in #154

maxim-belkin and others added 30 commits December 7, 2018 16:54
Signed-off-by: Maxim Belkin <[email protected]>
clarify objectives and key points
I see lots of contributions where none of this placeholder text has
been deleted, but also quite a lot where the first line has been deleted,
but the 'ext below' hasn't (exactly in accordance with the current
instructions).
Clarify wording of PR & issue templates
Use the same colors but apply 12.5% opacity.

Solution boxes use 30% opacity
Also, improve contrast of comments

Signed-off-by: Maxim Belkin <[email protected]>
This contribution adds reference to the web page of the lesson in the Carpentries and makes it easier for visitors who want to contribute to get familiar with the good first issue badge.
This adds a list of current maintainers to the README.md.
gold looks more like the yellow from the original badge
Database values for no language is an empty string instead of NULL in the Language table.
libcce and others added 28 commits August 15, 2020 07:23
Responds to #97
Thanks @jfrisina for finding this resource!
…ed-visually

Add SQL Join Types Explained Visually
…oin-explained-visually

Revert "Add SQL Join Types Explained Visually"
1. Very minor grammatical change (adding a comma to correct a run-on sentence).
2. Added a contextual phrase to align the use of the pronoun "we" with "librarians"
AVG(Citation_Count) needs to match AVG(Citation_Count) to be executed without an error.
Avg_Citation_Count needs to be rewritten
I changed the "has more than 4 citations" to "has more than 8 citations" because the dataset only has articles with "computer" that have 8 and 9 citations. if we edit this to have the citation count be greater than 8, then there will be a reason to add the the second filter. (The dataset returns 17 rows for the first condition (Subjects LIKE '%computer%'), and 11 rows when you add the second condition (Citation_Count > 8)
Change Challenge exercise related to FIltering
Changing the term "Prerequisites" (misleading phrase) to "Before You Start:"
Inspired by the Software Carpentry Course "Databases and SQL", Chapter 9. Creating and Modifying Data
https://github.com/swcarpentry/sql-novice-survey/edit/gh-pages/_episodes/09-create.md
I removed the callout as suggeted in #154
@github-actions
Copy link

🚫 The gh-pages branch is no longer editable 🚫

Thank you for your contribution. This lesson has migrated to use The Carpentries Workbench and the gh-pages branch is now automatically generated. This means in order to contribute, you will need to delete and re-fork this repository.

How to contribute

If you wish to contribute, you will need to use the following steps to delete, re-fork, and re-create your pull request (aka the burn it all down strategy):

  1. Save your edits on locally or in a scratch space.
  2. Delete your fork
  3. Create a new fork or use the "edit" button on the page you wish to edit.
  4. Apply your changes (NOTE The Workbench uses a different syntax. Here is a Transition Guide from Styles to Workbench for your reference).

Questions

If you have any questions or would like assistance, please contact @core-team-curriculum ([email protected]) or you can respond to this message.

@github-actions github-actions bot closed this Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.