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

[BUMP] Update dependency ember-template-lint to v6 (admin) - à revoir #8601

Closed
wants to merge 1 commit into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 5, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
ember-template-lint ^5.0.0 -> ^6.0.0 age adoption passing confidence

Release Notes

ember-template-lint/ember-template-lint (ember-template-lint)

v6.0.0

Compare Source

💥 Breaking Change
  • #​3029 Enable more recommended rules: no-action-on-submit-button, no-at-ember-render-modifiers, no-builtin-form-components, no-unnecessary-curly-parens, no-unnecessary-curly-strings, simple-modifiers (@​bmish)
  • #​3031 Replace 4-x-recommended config with 5-x-recommended (@​bmish)
  • #​3023 Drop support for Node 14, 16, 19 (@​Techn1x)
  • #​2913 Add additional elements to DOCUMENT_STRUCTURE_ROLES for no-invalid-role rule (@​mrloop)
  • #​2729 Add param to obsolete list in no-obsolete-elements rule (@​geneukum)
🏠 Internal
Committers: 4

Tracking issue: https://github.com/ember-template-lint/ember-template-lint/issues/2712


Configuration

📅 Schedule: Branch creation - "every 1 hour every weekday" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Never, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label Apr 5, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@xav-car
Copy link
Contributor

xav-car commented Apr 8, 2024

grosso modo

Changer pour des PixInput sur les 74 usages de l'Input ember
app/templates/authenticated/trainings/training/target-profiles.hbs
10:10 error Do not use the Input component. Built-in form components use two-way binding to mutate values. Instead, refactor to use a native HTML element. no-builtin-form-components

✖ 74 problems (74 errors, 0 warnings)

@yannbertrand
Copy link
Member

Voir aussi 1024pix/pix-ui#593

@yaf yaf force-pushed the renovate/admin-ember-template-lint-6.x branch from 8a7da07 to 95bcc38 Compare June 11, 2024 16:43
Copy link
Contributor Author

renovate bot commented Jun 11, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@yaf yaf force-pushed the renovate/admin-ember-template-lint-6.x branch 3 times, most recently from 65f1ccf to 477b43e Compare June 12, 2024 07:13
@yaf
Copy link
Member

yaf commented Jun 12, 2024

  • transformation des Input en PixInput
  • transformation des TextArea en PixTextearea
  • suppression des did-insert

Pour les Input et Textarea j'ai fait de manière mécanique, il y avait des attributs à modifier ?

Pour la suppression des did-insert, j'ai regardé rapidement ce qui avait été fait, et je ne suis tombé que sur des commits de @xav-car qui les supprime, tout simplement... Je n'ai pas l'impression qu'il y est un bout de code qui compense... Mais un truc a dû m'échapper.

Je viens de comprendre à quoi ça sert, et de comprendre que du coup, c'est dans le constructeur du composant que ça se passe...

Si quelqu'un qui connait le truc peu m'orienter, voir faire le taf, ça me va bien, je voulais juste filer un coup de main en passant par là.

@yannbertrand
Copy link
Member

Pour la suppression des did-insert, j'ai regardé rapidement ce qui avait été fait, et je ne suis tombé que sur des commits de @xav-car qui les supprime, tout simplement... Je n'ai pas l'impression qu'il y est un bout de code qui compense... Mais un truc a dû m'échapper.

Je crois me rappeler qu'il a déplacé le code correspondant dans les constructor, comme sur la PR Pix UI non ?

@yaf
Copy link
Member

yaf commented Jun 12, 2024

Oui, c'est ça. Mais je ne suis pas certain de savoir quel constructeur chercher sur cet exemple

image

C'est dans le fichier /admin/app/components/certifications/candidate-edit-modal.hbs

@yaf
Copy link
Member

yaf commented Jun 12, 2024

Enfin, si, j'ai trouvé le fichier, mais je ne comprends pas trop comment je peux « pousser » le focus sur cet input... 🤔

@yannbertrand
Copy link
Member

Enfin, si, j'ai trouvé le fichier, mais je ne comprends pas trop comment je peux « pousser » le focus sur cet input... 🤔

Ah oui gloups ! Peut être qu'il manque un argument à ajouter dans le PixInput ? :(

@yaf yaf force-pushed the renovate/admin-ember-template-lint-6.x branch 3 times, most recently from a91322d to f6b67fd Compare June 13, 2024 08:48
@renovate renovate bot changed the title [BUMP] Update dependency ember-template-lint to v6 (admin) [BUMP] Update dependency ember-template-lint to v6 (admin) - abandoned Jun 17, 2024
Copy link
Contributor Author

renovate bot commented Jun 17, 2024

Autoclosing Skipped

This PR has been flagged for autoclosing. However, it is being skipped due to the branch being already modified. Please close/delete it manually or report a bug if you think this is in error.

@yaf yaf force-pushed the renovate/admin-ember-template-lint-6.x branch 2 times, most recently from d917885 to 5000a48 Compare June 26, 2024 09:00
@yaf
Copy link
Member

yaf commented Jun 26, 2024

Cette PR comment à devenir compliqué à gérer ...

@yannbertrand
Copy link
Member

yannbertrand commented Jul 18, 2024

On en cause à l'occas' ? Voir comment on fait avancer ça... ou pas !

Je suis tombé sur cette PR qui peut nous intéresser ?

@yaf yaf force-pushed the renovate/admin-ember-template-lint-6.x branch from 31e9b4a to 147e1e6 Compare August 7, 2024 08:01
@yannbertrand yannbertrand changed the title [BUMP] Update dependency ember-template-lint to v6 (admin) - abandoned [BUMP] Update dependency ember-template-lint to v6 (admin) - à revoir Aug 30, 2024
@xav-car xav-car deleted the renovate/admin-ember-template-lint-6.x branch September 2, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants