If the heading of a previous release in the changelog has a link to a compare view, the Action will use that URL and add a link to the updated compare view to the new release notes.
It's hard to explain this with words. The attached screenshot gives you a good example.
This previously only worked, if the changelog had a "Unreleased" heading with a link in it.
Also kinda embarrassed by the state of the underlying CLI php code base.
It's currently a mess of "Action"-classes calling other "Action"-classes and passing the CLI options all the way down.
Have to find a good way to define all the “scenarios" and then isolate them into separate classes.
eg. "release notes passed, unreleased heading found, markdown file is empty: place notes at the end of the document”
@stefanzweifel been fighting this kind of thing too. one thing that has helped me personally is that when i need to print status messages... if i am not in "the cli app file" or 1 level of class deeper, aka my ./bin/whatever.php, then i have done it all wrong and stop.
this has helped me a lot because now i do a terminal app, and then like a terminal app... subcontroller?
so my main app will just be like DoThatStuff() new StuffDoer(this), and i do not allow cli related things to go any deeper.
Add comment