Teds Snapshot #50
Open
Teds Snapshot #50
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Alex-Jordan
reviewed
Aug 18, 2021
| <onesentence> | ||
| These notes constitute a self-contained sequence for teaching, | ||
| using an inquiry-based pedagogy, most of the traditional topics | ||
| of Calculus I, II, & III in classes of up to 40 students. |
Contributor
There was a problem hiding this comment.
As this is an XML file, it can't have a bare & character like this. You could change this to &, but I might recommend just using the word 'and'.
Contributor
|
I'm reviewing PRs on pretext-projects, including this PR. @rbeezer, @davidfarmer, do we have any minimal requirements for a new entry? Things I noticed with the entries here:
Is a new entry OK like this? @taterheadted Thanks for your submissions and I'm not complaining about the things above :) But I'm using your pull request as a basis for discussing these things, and if the group wants to have minimal requirements. |
Open
Contributor
|
I suggest:
a) either there is a link to the compiled document, or an explanation
for why a link cannot be provided.
b) the license has to be written in a complete form (so, "CC" is not
sufficient)
c) if the license suggests the project is open source,
then a link to the source must be provided.
…On Wed, 18 Aug 2021, Alex Jordan wrote:
I'm reviewing PRs on pretext-projects, including this PR. @rbeezer, @davidfarmer, do we have any
minimal requirements for a new entry? Things I noticed with the entries here:
* There are no links to see the projects in any form (no HTML, no source files, etc).
* The license is marked CC, but the @variant and @Version are left empty.
Is a new entry OK like this?
@taterheadted Thanks for your submissions and I'm not complaining about the things above :) But I'm
using your pull request as a basis for discussing these things, and if the group wants to have
minimal requirements.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS orAndroid.[AABTULAPTWRVMQJT34UC5YLT5P7DHA5CNFSM4KUVVIX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN
5WW2ZLOORPWSZGOGW4ZBNQ.gif]
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.