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(books): free-programming-books-pt_BR broken links #6955

Merged
merged 1 commit into from
Jul 21, 2022
Merged

fix(books): free-programming-books-pt_BR broken links #6955

merged 1 commit into from
Jul 21, 2022

Conversation

LuigiImVector
Copy link
Member

What does this PR do?

Improve repo

For resources

Description

Solved a problem of #6942, but...

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

They are books

Checklist:

  • Read our contributing guidelines
  • Search for duplicates.
  • Include author(s) and platform where appropriate.
  • Put lists in alphabetical order, correct spacing.
  • Add needed indications (PDF, access notes, under construction)

Follow-up

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

@LuigiImVector LuigiImVector self-assigned this Jul 20, 2022
@LuigiImVector LuigiImVector added the 🗣️ locale:pt Resources addressing "Portuguese / Brazilian" language label Jul 20, 2022
@davorpa davorpa added waiting for changes PR has been reviewed and changes/suggestions requested 🚀 ready to merge LGTM. Waiting for final approval and removed waiting for changes PR has been reviewed and changes/suggestions requested labels Jul 20, 2022
@davorpa davorpa added the urlchecker Checker workflow monitoring the links of resources has found some warnings. See logs and fix URLs. label Jul 21, 2022
@eshellman
Copy link
Collaborator

go team

@eshellman eshellman merged commit e902205 into EbookFoundation:main Jul 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eshellman eshellman eshellman left review comments

@davorpa davorpa davorpa approved these changes

Assignees

@LuigiImVector LuigiImVector

Labels
🚀 ready to merge LGTM. Waiting for final approval 🗣️ locale:pt Resources addressing "Portuguese / Brazilian" language urlchecker Checker workflow monitoring the links of resources has found some warnings. See logs and fix URLs.
Projects
None yet
Milestone
No milestone
Development

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