Improve build#317
Conversation
Artifacts were generated in src/, instead these go to target/ now.
These are useful when reading the build output to diagnose issues
Profiles were chosen based on github specific info. By default on a local machine Cypress should run without recording when 'mvn verify' is run. On CI Cypress will run with recording. Options can be overriden locally on in CI with 'mvn -P'.
Also, remove OS from matrix as there was only one option to begin with.
|
@adamretter Your draft PR contained a number of excellent commits. I would have pulled it in were it not for the conflict in Cypress versions (upgrade to 10 but we are already on 14). Plus, I wanted to add a few tweaks on top of that. |
|
As written on top, these are cherry picks from your PR, most of it was very useful. |
|
@line-o @dizzzz Im very sad to see that my input has been ignored. We were not on Cypress 14 until my concerns about that were completely ignored and that change made anyway. Now without any notice to me, and without even engaging in a conversation with me, my work has been taken, and used, whilst duplicating effort I already made. These actions have been revealing. It is now very clear to everyone outside of this project, that this is not at all a collaborative project, nor does it welcome external contributions. The invite publicly given to me at MarkupUK to collaborate on eXist-db Apps was apparently a hollow piece of face saving publicity. I regret spending my time on this project, it's now clear to me it's been a waste of my time, and that my repeated attempts to engage and collaborate have fallen on already deaf ears. |
|
@adamretter your comments and contributions are certainly NOT ignored! That is not fair to anyone. As you state it is a community based project, and in the community there are more voices, opinions, insights, experiences and solutions, not only yours. So in the community call we took all options seriously, we investigated things, we considered all options and in the end we took the way the community thought it was the best way to go. IMO that is the way it should go; 'one voice' should not have a veto or something like that. |
5 participants
Cherry picked commits from #313 + a minor tweak