Skip to content

Issue #76#94

Merged
seidewitz merged 30 commits intomaintenancefrom
Issue-#76
Oct 21, 2020
Merged

Issue #76#94
seidewitz merged 30 commits intomaintenancefrom
Issue-#76

Conversation

@abstratt
Copy link
Copy Markdown
Contributor

@abstratt abstratt commented May 23, 2020

This is work-in-progress for Issue #76, please do not merge, sharing partial progress for feedback.

  • first stab: not touching Eclipse bundles yet (no Tycho yet)
  • removing jars that can now be obtained via Maven
  • removing Eclipse configuration files that are now automatically generated when imported as Maven projects
  • some tests were relying on both JUnit 4 and 5 - using JUnit 5 only now, which required some changes due to non-backward compatible changes in JUnit 5
  • renamed test source path for the test module
  • enabled CI on Travis CI (based on a fork of this repository under my own GitHub user as this repository does not have Travis enabled yet), but we could move elsewhere. Note that there is was one test failing, but that was the case already in master (this change from January changed how resolveModelFile() works for files with errors, so it now returns a UnitDefinition even if there are errors, as the test expects expected a null UnitDefinition to be returned, which I guess is - and maybe already was - an invalid expectation). I changed the test so it expects a non-null result.

abstratt added 2 commits May 23, 2020 18:55
- first stab: not touching Eclipse bundles yet (no Tycho yet)
- removing jars that can now be obtained via Maven
- removing Eclipse configuration files that are now automatically generated when imported as Maven projects
- some tests were relying on both JUnit 4 and 5 - using JUnit 5 only now
- renamed test source path for the test module
@abstratt abstratt changed the title Issue #76 (WIP) Issue #76 Jul 6, 2020
@abstratt
Copy link
Copy Markdown
Contributor Author

abstratt commented Jul 6, 2020

@seidewitz This is now ready for review. The only thing I didn't do yet is to generate the dist jars with their final names, I am ATM generating them using a "new-" prefix, so they do not overwrite yet the dist files you created by hand.

Oh, the other thing I didn't enable was enabling CI on Travis (as I did in my fork under my own user name) and pushing the binaries to a Maven repository like Bintray (as I did with my own projectsd) because you probably should (must?) be the one enabling Travis-CI for the ModelDriven organization on GitHub and crfeating the project on Bintray.

@abstratt abstratt linked an issue Jul 31, 2020 that may be closed by this pull request
@abstratt
Copy link
Copy Markdown
Contributor Author

@seidewitz Please see final changes just committed, which include sections for the README.md file. Also, this still does not clobber the existing archives you have been creating by hand, but instead produces the packages using a -new suffix, so you can compare them easily (once the results are deemed valid, we should change the pom.xml file to remove the use of the prefix (look for notes such as TODO-RC: remove new- prefix once validated). Please see above: #94 (comment)).

Any further questions, please let me know.

@seidewitz seidewitz marked this pull request as ready for review October 20, 2020 18:51
@seidewitz seidewitz merged commit 6eb2769 into maintenance Oct 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adopt Maven+Tycho to allow automated builds

2 participants