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.

Sign up for GitHub

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

Jump to bottom

fix: preserve title - author format #7032

Merged
merged 5 commits into from
Aug 22, 2022
Merged

fix: preserve title - author format #7032

merged 5 commits into from
Aug 22, 2022

Conversation

davorpa
Copy link
Member

@davorpa davorpa commented Aug 22, 2022 •
edited
Loading

What does this PR do?

Improve repo

For resources

Description

How do we know it's really free?

For book lists, is it a book? For course lists, is it a course? etc.

Checklist:

  • Read our contributing guidelines.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Used an informative name for this pull request.

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

@davorpa davorpa added 🗣️ locale:en Resources addressing "English" language 🗣️ locale:es Resources addressing "Spanish / español" language 🗣️ locale:ja Resources addressing "Japanese / 日本語" language 🗣️ locale:si Resources addressing "Sinhala / සිංහල" language labels Aug 22, 2022
@davorpa davorpa self-assigned this Aug 22, 2022
@davorpa davorpa marked this pull request as ready for review August 22, 2022 03:38
@eshellman
Copy link
Collaborator

lgtm

@eshellman eshellman added the 🚀 ready to merge LGTM. Waiting for final approval label Aug 22, 2022
@davorpa davorpa merged commit d9d3c34 into main Aug 22, 2022
davorpa referenced this pull request Aug 22, 2022
@davorpa
Merge branch 'main' into normalize-authoring-format
31dc7bb
@davorpa davorpa mentioned this pull request Aug 22, 2022
Merged
5 tasks
@davorpa davorpa deleted the fix-spaces-arround-title-and-author-delimiter branch September 28, 2022 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@davorpa davorpa

Labels
🚀 ready to merge LGTM. Waiting for final approval 🗣️ locale:en Resources addressing "English" language 🗣️ locale:es Resources addressing "Spanish / español" language 🗣️ locale:ja Resources addressing "Japanese / 日本語" language 🗣️ locale:si Resources addressing "Sinhala / සිංහල" language
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davorpa @eshellman