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

Add the first julia cource #6110

Merged
merged 2 commits into from
Oct 10, 2021
Merged

Conversation

nagayev
Copy link
Contributor

@nagayev nagayev commented Oct 9, 2021

What does this PR do?

Add resource

For resources

Description

Why is this valuable (or not)?

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
  • 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!

@davorpa davorpa self-requested a review October 9, 2021 21:07
Copy link
Member

@davorpa davorpa left a comment

Choose a reason for hiding this comment

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

It's a good moment to make some changes. When done, request for a review again using GitHub 🔃 button.

courses/free-courses-ru.md Outdated Show resolved Hide resolved
@davorpa davorpa added the waiting for changes PR has been reviewed and changes/suggestions requested label Oct 9, 2021
Co-authored-by: David Ordás <3125580+davorpa@users.noreply.github.com>
@davorpa davorpa added 🚀 ready to merge LGTM. Waiting for final approval and removed waiting for changes PR has been reviewed and changes/suggestions requested labels Oct 10, 2021
@nagayev
Copy link
Contributor Author

nagayev commented Oct 10, 2021

@davorpa Why you have add ready to merge label, but not merge it?
I'm just interesting

@davorpa
Copy link
Member

davorpa commented Oct 10, 2021

Why you have add ready to merge label, but not merge it? I'm just interesting

It is our internal guidelines to triage the contributions. I haven't write access rights

@nagayev
Copy link
Contributor Author

nagayev commented Oct 10, 2021

@davorpa okey, thank you for quick response

@eshellman
Copy link
Collaborator

Triage and review volunteers are essential to the effective running of this repo. @nagayev consider helping us py reviewing Russian submissions.

@eshellman eshellman merged commit ee47b42 into EbookFoundation:master Oct 10, 2021
@nagayev
Copy link
Contributor Author

nagayev commented Oct 10, 2021

@eshellman this project is interesting and useful for me.
So I'm agree.

@nagayev
Copy link
Contributor Author

nagayev commented Nov 3, 2021

@eshellman Hello. Is your invitation valid?
I thought I would become a member of your organization.

@davorpa
Copy link
Member

davorpa commented Nov 3, 2021

Take a look at #6373 too 😉

@nagayev
Copy link
Contributor Author

nagayev commented Nov 3, 2021

@davorpa thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 ready to merge LGTM. Waiting for final approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants