Skip to content

Conversation

@reckart
Copy link
Contributor

@reckart reckart commented Apr 4, 2024

  • Moved Wicket JQuery / Kendo into subfolders
  • Integrated Wicket JQuery / Kendo modules into the build process
  • Remove redundancies from the wicket-jquery-ui-parent pom - everything necessary should be in the wicketstuff parent POM already
  • Change groupIds to match wicketstuff (not for 9.x)
  • Change artifactIds to match wicketstuff conventions (not for 9.x)
  • Change module names to match wicketstuff conventions (not for 9.x)
  • Change package names to match wicketstuff (not for 9.x)

Sebastien Briquet and others added 30 commits July 12, 2017 14:16
Kendo UI: Added factory methods CommandColumn#of(...)
Kendo UI: **API BREAK** LinkPropertyColumn is now abstract, callback-url should now be supplied. 'datable' and 'button' arg have been removed from ctors
Kendo UI: added CommandButton#toJSONObject()
Kendo UI: added ToolbarButton#toJSONObject()
Added KendoEffect 'flip'
- Adjust groupId
- Integrated Wicket JQuery/Kendo modules into the build process
- Adjusted versions
- Moved new modules as subfolders under wicket-jquery-ui-parent to align with the structure of other submodules
@solomax
Copy link
Contributor

solomax commented Apr 5, 2024

@reckart I would modify description and remove
Change groupIds to match wicketstuff
and
Change artifactIds to match wicketstuff conventions

otherwise we will lose backward compatibility :)

@reckart
Copy link
Contributor Author

reckart commented Apr 5, 2024

Mind if we keep the old groupIds, we have a release that contains multiple groupIds. Do you know if that has implications when deploying to Maven Central / Nexus?

@solomax
Copy link
Contributor

solomax commented Apr 5, 2024

Originally it was deployed to https://oss.sonatype.org (same as wicketstuff)

After merge I can run release process, and we will check if there will be any issues :)

@reckart reckart force-pushed the refactoring/850-Merging-wicket-jquery-ui-into-wicketstuff-core-9.x branch from 1f7a8ed to 0c79fc5 Compare April 12, 2024 16:47
@reckart reckart force-pushed the refactoring/850-Merging-wicket-jquery-ui-into-wicketstuff-core-9.x branch from 0c79fc5 to f315d12 Compare April 12, 2024 16:50
@reckart reckart force-pushed the refactoring/850-Merging-wicket-jquery-ui-into-wicketstuff-core-9.x branch from f315d12 to 091ef57 Compare April 12, 2024 17:08
- Use the old groupId for the Wicket JQuery UI / Kendo modules for backwards compatibility
@reckart reckart force-pushed the refactoring/850-Merging-wicket-jquery-ui-into-wicketstuff-core-9.x branch from 091ef57 to 6d27a30 Compare April 12, 2024 17:29
@reckart
Copy link
Contributor Author

reckart commented Apr 12, 2024

Ok, I have change the groupIds of the Wicket JQuery UI / Kendo modules back to the old one for backwards compatibility. Anything more to do before merging?

@reckart reckart marked this pull request as ready for review April 12, 2024 20:52
@reckart reckart self-assigned this Apr 12, 2024
@reckart reckart added this to the 9.18.0 milestone Apr 12, 2024
@solomax solomax merged commit f67b4bb into wicketstuff:wicket-9.x Apr 15, 2024
@solomax
Copy link
Contributor

solomax commented Apr 15, 2024

Merged, I'll try to check everything locally ASAP!

@solomax
Copy link
Contributor

solomax commented Apr 16, 2024

@reckart SNAPSHOT succesfully deployed to https://oss.sonatype.org/

@solomax
Copy link
Contributor

solomax commented Jun 4, 2024

Can someone suggest how to get filtered shortlog?

with this 1000+ commits PR usual shortlog is useless :((((

@reckart
Copy link
Contributor Author

reckart commented Jun 4, 2024

I don't find a way at the moment of excluding a certain commit or range of commits from the shortlog. My idea would be to check the authors of the original wicket-jquery repo and then manually remove most/all the authors from that from the shortlog for the current releases, assuming that there is no significant relevant overlap between the authors of the two repos or the contributions they have made to the two repos in the relevant time spans.

@reckart
Copy link
Contributor Author

reckart commented Jun 4, 2024

Maybe: git shortlog wicketstuff-core-9.17.0..wicketstuff-core-9.18.0 --not 6d27a309d9d94bf1355ce5abeaacff3ce9e9dfbd - the hash is the commit where we merged wicket-jquery

@solomax
Copy link
Contributor

solomax commented Jun 4, 2024

Thanks a million @reckart !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants