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

Improve Japanese resource #5550

Merged
merged 7 commits into from
Sep 20, 2021
Merged

Improve Japanese resource #5550

merged 7 commits into from
Sep 20, 2021

Conversation

SuzukiKatsuma
Copy link
Contributor

What does this PR do?

Improve repo

For resources

Description

  • When I was editing Japanese resources, I noticed that many of them linked to "http" when they should have linked to "https". So I fixed those.
    Similarly, I fixed the "/" and "index.html" at the end of the links.
    I checked carefully to make sure that this would not break the links.
  • Old links found during the above work have been brought up to date.
    The names had changed, so I also reordered them.

Why is this valuable (or not)?

"free-programming-books-ja.md" will now follow the guidelines more closely than before.

How do we know it's really free?

I didn't add any books.

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

I didn't add any 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)

Followup

  • Check the output of Travis-CI for linter errors!

Those that work correctly are fixed from "http" to "https".
Also, some things that can be omitted have been omitted.
@SuzukiKatsuma
Copy link
Contributor Author

Fixed an error in alphabetical order.

@SuzukiKatsuma
Copy link
Contributor Author

SuzukiKatsuma commented Sep 19, 2021

Sorry, I missed one 😅



#### AngularJS

> :information_source: See also … [Angular](#angular)
> :information_source: 関連項目 - [Angular](#angular)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ellipsis is part of crosslinks proposal #5535

Suggested change
> :information_source: 関連項目 - [Angular](#angular)
> :information_source: 関連項目 … [Angular](#angular)



#### Angular

> :information_source: See also … [AngularJS](#angularjs)
> :information_source: 関連項目 - [AngularJS](#angularjs)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ellipsis is part of crosslinks proposal #5535

Suggested change
> :information_source: 関連項目 - [AngularJS](#angularjs)
> :information_source: 関連項目 … [AngularJS](#angularjs)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment welcome of crosslinks syntax. this gives us another option.

@davorpa
Copy link
Member

davorpa commented Sep 20, 2021

Thanks for translating See also 🤗

@eshellman
Copy link
Collaborator

Thanks!

@eshellman eshellman merged commit e639960 into EbookFoundation:master Sep 20, 2021
@SuzukiKatsuma SuzukiKatsuma deleted the improve-Japanese-resource branch September 20, 2021 17:31
@SuzukiKatsuma SuzukiKatsuma restored the improve-Japanese-resource branch September 20, 2021 17:32
@SuzukiKatsuma SuzukiKatsuma deleted the improve-Japanese-resource branch September 20, 2021 19:02
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