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

Added AutoIt, language updates and support #5554

Merged
merged 7 commits into from
Sep 27, 2021
Merged

Added AutoIt, language updates and support #5554

merged 7 commits into from
Sep 27, 2021

Conversation

kernelwernel
Copy link
Contributor

@kernelwernel kernelwernel commented Sep 25, 2021

What does this PR do?

Add resources, Improve repo

For resources

Description

  • This PR is basically just 2 things at once. I added language support for french and contributed some miscellaneous stuff on the available languages which had outdated hyperlinks to other translation files of the same .md file.
  • added french translation of CONTRIBUTING.md

(I admit I kinda abused google translate for the translation but it should be readable and understandable to any french speaking person. I'm aware if I just directly copy pasted everything lazily from google translate then there will be a lot of issues or just generally wrong grammar, so I corrected/guided the faulty translations that were wrong which took me more than an hour.)

  • changed small grammatical stuff in the main CONTRIBUTING.md file
  • fixed the markdown section links in the vietnamese CONTRIBUTING-vn.md file
  • Included newer language files in the HOWTO.md header files of non-english translations
  • added AutoIt language playlist in courses and the docs in their appropriate sections

Why is this valuable (or not)?

I believe AutoIt is an extremely useful tool and language for windows users, as it can practically do anything when it comes to automation. It can basically do the same things as AutoHotKey but Its syntax is harder to learn (imo) than AutoIt and should serve as an alternative to AHK.

How do we know it's really free?

It's the official documentation website for AutoIt. It does not require anything other than an internet connection.

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

I added both a course and a book.

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!

@kernelwernel
Copy link
Contributor Author

Sorry about the weird changes I've made in free-courses-en.md and free-programming-books.md, I'm not entirely sure how I even managed to change all asterisks into hyphens for 250+ lines. But anyway, the PR should be good now.

@davorpa
Copy link
Member

davorpa commented Sep 26, 2021

Thanks for explain huge changes minutely 😄

Sorry about the weird changes I've made in free-courses-en.md and free-programming-books.md, I'm not entirely sure how I even managed to change all asterisks into hyphens for 250+ lines. But anyway, the PR should be good now.

I suggest to disable editor linter addons like vscode markdown-all-in-one. We use one built on top of @remarkjs.

I see you could solve it in the 5th commit 👍.


If you accept an advice, try to don't overscope PRs/commits 😉... It makes hard to review it, especially when mixing additions and deletions of if implies many files.

  • Add AutoIt resources
  • Translate CONTRIBUTE.md to French
  • Synchronize language list in HOWTO's

Making small commits, you get finer and easier control over revert / reset to previous states.

If you want submit more than one at same time, making branches (one per PR) instead work on master is the best option. Fetch and checkout them from upstream master.

git fetch --all
git checkout -b <branch_name> upstream/master
#... let's code, commit and then upload to your fork...
git push -u origin <branch_name>

This has also the advantage of unnecesary extra merge commits/conflicts by your side (e.g. commits 2nd a8f4bff, 4th 6f60e71) 🚀

@kernelwernel
Copy link
Contributor Author

kernelwernel commented Sep 26, 2021

I see, thanks for the explanation. I'll try to be a bit more careful and organised with my PRs
next time.

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.

Thanks for complete #5024 and #5454.

Some minor changes 😉 ⬇️

Could you find some other interesting AutoIt resources in both listings (books/course)? We try to avoid single items sections especially in these two which are too big (see #5092)

Thanks in advance.

HOWTO.md Show resolved Hide resolved
courses/free-courses-en.md Show resolved Hide resolved
@eshellman
Copy link
Collaborator

Formidable!

@eshellman eshellman merged commit e629db6 into EbookFoundation:master Sep 27, 2021
davorpa added a commit to davorpa/free-programming-books that referenced this pull request Sep 27, 2021
eshellman pushed a commit that referenced this pull request Sep 27, 2021
@kernelwernel kernelwernel mentioned this pull request Sep 29, 2021
5 tasks
davorpa referenced this pull request Nov 20, 2021
eshellman pushed a commit that referenced this pull request Feb 3, 2022
* docs(contributing): remove trailing spaces

* docs(contributing): Apply headings `2-1` empty lines lint rule

* docs(contributing): Apply headings `2-1` empty lines lint rule

forgotten in previous commit: e83220a

* docs(contributing): Move anchor alias to inside list items

... there where an anchor alias `<a name="...">` is used near to each of those list items

This solves markdownlint `MD032/blanks-around-lists` issue

* docs(contributing): Homogenize `1` blank line in nutshell enumeration

* docs(contributing): Homogenize `1` blank line inside Formatting section

* docs(contributing): Ensure `4-spaces` indent `.editorconfig` param

* docs(contributing): Homogenize format & indent of code blocks

- Format using fenced style.
- Well indent in those placed inside list enumerations.
- Simulate `Good` - `Bad` monospace double colon indent inside each one.

- Examples highlighting as `text-plain`. The `md, markdown` style doesn't work very well since blocks has mixed syntax.
- `check_urls` code block highlighted as `properties` style (key-value pair).

Solves markdownlint rules:
- MD040/fenced-code-language: Fenced code blocks should have a language specified
- MD046/code-block-style: Code block style [Expected: indented; Actual: fenced]

* docs(contributing): Ensure `4-spaces` indent `.editorconfig` param

Forgotten in commit: 31c776c

* docs(contributing): Homogenize format & indent of code blocks

- Simulate `Good` - `Bad` monospace double colon indent inside each one.

Revert commit f9dc719 only in this file after check non-needs in a deployed web snapshot

* docs(contributing): Use "consistent" unordered list mark

Solves markdownlint rules:
MD004/ul-style: Unordered list style "consistent"

* docs(contributing): Solve `no-space-in-*` lint rules

markdownlint rules:
MD037/no-space-in-emphasis: Spaces inside emphasis
MD038/no-space-in-code: Spaces inside code span elements
MD039/no-space-in-links: Spaces inside link text

* docs(contributing): Move some punctuation just before any closing brackets after it

* docs(contributing): Hack highlight IDE/Editors headings+HTML

It apply the hack to highlight in IDE/Editors those headings with an anchor alias just before itself

* docs(contributing): Homogenize Pull Request acronym

* docs(contributing): Fix `GitHub` trademark typos

* docs(contributing): ALLCAPS

* docs(contributing): Git trademark

* docs(contributing): YouTube trademark

* docs(contributing): ISBN acronym

* fix: flag emoji code `en` > `us`

Resolves d775cb3#commitcomment-59537594

#5831: `:en:` emoji flag markup doesn't work. Use `:us:` :us: or `:gb:` :gb:

d775cb3#diff-b35eb40c148657d4ea1361c44496e11c5625f484a4c6040c022770e88cbcfdccR8-R10

* docs(contributing): Fix fa_IR broken anchors

* docs(contributing): filenames in monospace font

* docs(contributing): Homogenize `check_urls=file_to_check`

* docs(contributing): literals/markup texts in monospace font

- `et al.`
- File extensions `.md`
- Protocols: `http`, `https`...
- Notes: LeanPub, require email, StackOverflow, in process, archived

trailing dots in that lines

* docs(contributing): linkify wayback machine

* docs(contributing): Fix `Leanpub` trademark typos

* chore: Rename `master` to `main` #blacklivesmatter

Resolves #5803

* docs(contributing): fix broken ref link to `programming-playgrounds`

Fix over #5449 and commit 3c42c36

(cherry picked from commit 59ebfe3)

* docs(contributing): fix repo urls on initial FR translation

- `ElivreFoundation` -> `EbookFoundation`
- `free-programming-livres` ->`free-programming-books`

PR: #5554
Commit: e629db6

Affected lines: 
- 18bb745#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R4 // e629db6#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R4
- 18bb745#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R7 // e629db6#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R7
- 18bb745#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R13 // e629db6#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R13
- 18bb745#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R45 // e629db6#diff-d6fa57ebb070f7f05f8c686a2fe92110652f48886774460ffee08d4be4cd5f27R45

* docs(contributing): use relative urls EN

Via #6614

* docs(contributing): greek. place translations link in right place

Via #6614

* Merge #6631 upstream/main commit into docs/contributing/sync/1

* Add repo badges (see #5983)

(cherry picked from a16a79b solving merge conflict)

* Merge #6662 upstream/main commit into docs/contributing/sync/1

* Apply centering proposed at #5983 but not implemented in #6631

(cherry picked from commit f36fd03)

* format(contributing): Bold 5th nutshell equal zh

* Merge #6677 upstream/main commit into docs/contributing/sync/1

fix(docs): Use Kramdown processor (#6677)
* Update _config.yml
* add markdown="1"
* update url
* complete removing spaces?
* explicitly use kramdown

(cherry picked from commit de2f319)
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