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

Titlecase for episode and section titles #165

Merged
merged 16 commits into from Dec 7, 2022

Conversation

anenadic
Copy link
Collaborator

@anenadic anenadic commented Dec 5, 2022

Improved consistency of titlecase use for episode and section titles.

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

❌ DANGER ❌

This pull request has modified workflows that created output. Close this now.

@anenadic anenadic changed the title Update 01-introduction.md Improved consistency of titlecase use for episode and section titles Dec 5, 2022
@anenadic anenadic changed the title Improved consistency of titlecase use for episode and section titles Titlecase for episode and section titles Dec 5, 2022
Even though we are not mentioning Bloom's taxonomy explicitly, highlighting different learning verbs/learning outcomes may be beneficial to show how different they can be (which is mentioned in Instructor Training too).
@anenadic anenadic marked this pull request as draft December 5, 2022 13:41
github-actions bot pushed a commit that referenced this pull request Dec 6, 2022
@anenadic anenadic marked this pull request as ready for review December 7, 2022 15:16
github-actions bot pushed a commit that referenced this pull request Dec 7, 2022
@zkamvar
Copy link
Contributor

zkamvar commented Dec 7, 2022

DANGER

This pull request has modified workflows that created output. Close this now.

Oh this is interesting. I'm going to investigate why this is happening.

@tobyhodges
Copy link
Member

Thank you @anenadic! @zkamvar I'm going to ignore the warning and merge this, as the diff shows no changes to the workflow files.

@tobyhodges tobyhodges merged commit eb20a4a into main Dec 7, 2022
github-actions bot pushed a commit that referenced this pull request Dec 7, 2022
Auto-generated via {sandpaper}
Source  : eb20a4a
Branch  : main
Author  : Toby Hodges <tobyhodges@carpentries.org>
Time    : 2022-12-07 19:20:46 +0000
Message : Merge pull request #165 from carpentries/anenadic-title-capitalisation

Titlecase for episode and section titles
github-actions bot pushed a commit that referenced this pull request Dec 7, 2022
Auto-generated via {sandpaper}
Source  : c490356
Branch  : md-outputs
Author  : GitHub Actions <actions@github.com>
Time    : 2022-12-07 19:23:18 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : eb20a4a
Branch  : main
Author  : Toby Hodges <tobyhodges@carpentries.org>
Time    : 2022-12-07 19:20:46 +0000
Message : Merge pull request #165 from carpentries/anenadic-title-capitalisation

Titlecase for episode and section titles
zkamvar added a commit to carpentries/sandpaper that referenced this pull request Dec 16, 2022
The pr-recieve and pr-comment workflows now use concurrencey
<https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#concurrency>
to cancel in-progress workflow runs if a new workflow run is started.
This will prevent situations like
<carpentries/lesson-development-training#165 (comment)>
from happening.

This PR will fix #374
@sstevens2 sstevens2 deleted the anenadic-title-capitalisation branch March 19, 2023 14:39
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.

None yet

3 participants