merge release 6 - Githubissues

zicht / messages-bundle

Library - ZichtMessagesBundle - Edit your translations in the database

MIT License

1 stars 0 forks source link

merge release 6 #49

Closed YuenZicht closed 1 year ago

YuenZicht commented 1 year ago

Followup of PR https://github.com/zicht/messages-bundle/pull/45

Separate branch/PR as i cannot merge directly

will be tagged as 7.1.1

YuenZicht commented 1 year ago

spam

YuenZicht commented 1 year ago

Ik besef het nu pas, maar zou je voor de 6.x branch nog even het changelog kunnen aanpassen in een aparte PR voor die branch?

zat ik wel over na te denken, alleen de bug is gefixed in 5.3.2. is het nodig dat we de changelog aanpassen voor elke release?

VMeijer-Zicht commented 1 year ago

Ik besef het nu pas, maar zou je voor de 6.x branch nog even het changelog kunnen aanpassen in een aparte PR voor die branch?

zat ik wel over na te denken, alleen de bug is gefixed in 5.3.2. is het nodig dat we de changelog aanpassen voor elke release?

Ja, het klinkt een beetje als 'overkill', maar ik vermoed dat het voor de duidelijkheid zo wordt gedaan. Zo is het bijvoorbeeld genoteerd voor release 7.0.1 in deze bundel:

## 7.0.1 - 2023-01-19
### Fixed
- Forward merge from 6.0.2

YuenZicht commented 1 year ago

Ja, het klinkt een beetje als 'overkill', maar ik vermoed dat het voor de duidelijkheid zo wordt gedaan.

Ah prima, heb even een kleine PR gemaakt voor vorige release: https://github.com/zicht/messages-bundle/pull/50 Ik moet dit wel taggen als 6.0.4, aangezien 6.0.3 al bestaat.