search

Igalia / team

Web service to gather and analyse skills of members of a team
MIT License
3 stars 3 forks source link

Changed wording in the assessment to be more neutral #114

Closed alex-voodoo closed 9 months ago

alex-voodoo commented 9 months ago

This patch addresses a discussion about better wording in the self-assessment form.

alex-voodoo commented 9 months ago

TIL that it is better to send the DB migration in a separate PR. The reason is: the DB schemas in main and stable have diverged, and the migration that targets main may not work for stable (the migrations refer to their parents). I had to fix this on stable because the deployment failed. The safer way is: first land the change to the models into main, then cherry-pick it into the stable, then generate migrations in both branches and commit independently.