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 yarn in tool-versions #2050

Closed
wants to merge 1 commit into from

Conversation

Matsa59
Copy link

@Matsa59 Matsa59 commented Jan 14, 2022

Hello,

Tres heureux de voir un projet opensource Francais en elixir.

Je vais surement un peu digger dans le projet pour comprendre comment tout fonctionne. En partant d'une fresh install j'ai eu une petite erreur avec yarn (qui manquait a mon system).

Je vais surement faire des petits PR au fur et a mesure de ma lecture du code si ca ne derange pas.

J'en profite pour vous demander si vous avez des good first issue a recommander.


Changes:

  • add yarn 1.22.17 in .tool-versions
  • update the test for a better message format

  1) test make sure LiveView client assets are up to date (TransportWeb.BuildTest)
     test/build_test.exs:66
     Your javascript package for phoenix_live_view is out of date.

     Please update it with:

         cd apps/transport/client && yarn upgrade phoenix_live_view

* add yarn 1.22.17 in `.tool-versions`
* update the test for a better message format

```

  1) test make sure LiveView client assets are up to date (TransportWeb.BuildTest)
     test/build_test.exs:66
     Your javascript package for phoenix_live_view is out of date.

     Please update it with:

         cd apps/transport/client && yarn upgrade phoenix_live_view

```
@Matsa59 Matsa59 requested a review from a team as a code owner January 14, 2022 02:53
@thbar
Copy link
Contributor

thbar commented Jan 14, 2022

Merci @Matsa59! On va en parler avec l'équipe 😄

Le test sur la MAJ des assets comporte par ailleurs un bug qui fait que ça ne suffit pas (#1938), pour info!

@thbar
Copy link
Contributor

thbar commented Jan 24, 2022

@Matsa59 on te remercie à nouveau de ta PR.

Dans l'immédiat toutefois après avoir échangé avec l'équipe :

  • Pour yarn on va préférer une installation hors asdf, et n'utiliser asdf que sur des plugins plus importants, qu'on estime moins vulnérables à des attaques. En particulier aujourd'hui on ne peut pas "pinner" facilement des versions de plugins, ce qui rend un "take-over" relativement aisé (voir Versioning of plugins asdf-vm/asdf#166). En n'utilisant que des plugins mieux maintenus, on réduit la surface d'attaque un petit peu (même si ça reste une préoccupation).
  • Pour d'autres contributions, on a réalisé que nous n'avons pas de "contributor license agreement" correct, et que cela pourrait nous mettre en difficulté si nous étions contraints de modifier la license pour des raisons variées dans le futur (j'ai déjà eu à suivre des cas où ça s'est produit, et ça a pu être compliqué). Du coup on ne va pas pouvoir non plus prendre de contribution directe sur le repo non plus, malheureusement. Si cela change (on y réfléchira), on te recontactera ; désolé dans l'immédiat !

Une idée (qui ne t'intéressera pas forcément, mais je la jette quand même) serait de contribuer en utilisant des données et en partageant des travaux sur un blog de ton côté ? (je sais que je compte faire cela à titre personnel dans le futur).

Merci malgré tout !

@thbar thbar closed this Jan 24, 2022
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.

2 participants