FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Alternation (OR) | by vplentinax · Pull Request #473 · javascript-tutorial/es.javascript.info · GitHub

/ es.javascript.info Public

Alternation (OR) | - #473

Merged
joaquinelio merged 10 commits into
javascript-tutorial:masterfrom
vplentinax:or
Feb 3, 2021
ConversationCommits (10)ChecksFiles changed
Merged

Alternation (OR) |#473
joaquinelio merged 10 commits into
javascript-tutorial:masterfrom
vplentinax:or

Conversation

vplentinax commented Jan 17, 2021

Copy link
Copy Markdown
Contributor

No description provided.

javascript-translate-bot requested a review from a team January 17, 2021 18:25
javascript-translate-bot mentioned this pull request Jan 17, 2021
Open
joaquinelio requested changes Jan 17, 2021
View reviewed changes

joaquinelio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lo del escape...

Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/solution.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/task.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/task.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/task.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/task.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/article.md

javascript-translate-bot commented Jan 17, 2021

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

vplentinax and others added 5 commits January 19, 2021 17:05
Update 9-regular-expressions/13-regexp-alternation/03-match-quoted-st…
1325296 
…ring/solution.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Update 9-regular-expressions/13-regexp-alternation/03-match-quoted-st…
3bb53ad 
…ring/task.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Update 9-regular-expressions/13-regexp-alternation/03-match-quoted-st…
bef3bd5 
…ring/task.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Update 9-regular-expressions/13-regexp-alternation/03-match-quoted-st…
cdbcf15 
…ring/task.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
Update 9-regular-expressions/13-regexp-alternation/03-match-quoted-st…
7911a22 
…ring/task.md

Co-authored-by: joaquinelio <joaquinelio@gmail.com>
joaquinelio requested changes Jan 20, 2021
View reviewed changes

joaquinelio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

comillas y espacios,
hay varios pero se me cruzan los ojos tengo que andar + y - zoom, sorry la review deficiente

sugiero una linea que creo que es una mejora conceptual,
no la mando al inglés porque ya estoy molestando demasiado y la ultima Ilya la aprobo de mala gana (y no era buena, en esp quedo mejor)
Claro que si cambia en algo el concepto no lo cambio en ESP si Ilya no lo acepta en ENG, no es el caso.

este es un art corto pero arduo, buen aporte al repo
Ya no tengo objeciones conceptuales, me gustaria que hicieras el merge aceptes o no los nuevos cambios

Comment thread 9-regular-expressions/13-regexp-alternation/01-find-programming-language/solution.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/01-find-programming-language/solution.md Outdated
Comment thread 9-regular-expressions/13-regexp-alternation/03-match-quoted-string/task.md Outdated

javascript-translate-bot commented Jan 20, 2021

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

joaquinelio merged commit 7ea15bd into javascript-tutorial:master Feb 3, 2021

javascript-translate-bot commented Feb 3, 2021

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #17 🎉 🎉 🎉

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

joaquinelio joaquinelio requested changes

Assignees

No one assigned

Labels

changes requested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Back | FazBrowse Home | New Git URL