Skip to content

Conversation

@marko-bekhta
Copy link
Member

LGPL info is a bit outdated 🫣 🙂

Comment on lines +62 to +65
/*
* SPDX-License-Identifier: Apache-2.0
* Copyright Red Hat Inc. and Hibernate Authors
*/
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +46 to +47
Each Hibernate project includes a set of build plugins to format and verify the code style of the sources.
Please refer to the particular project contributing guide for more details.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uh, okay, but Hibernate ORM for example redirects here:

https://github.com/hibernate/hibernate-orm/blob/main/CONTRIBUTING.md#guidelines

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😭 😖 infinite loop

though I have some slight doubts that everyone follows these instructions...
otherwise we'd be getting all new PRs with the old LGPL headers 😕

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really care about instructions for the headers since they're checked automatically on build, but the code style, IIRC, is not always checked in Hibernate ORM, and anyway it's convenient to have instructions on how to set up the IDE...

But those instructions would definitely be better to have in CONTRIBUTING.md, that I can agree with. The question is... what should those instructions be, if we archive hibernate-ide-codestyles :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hibernate/hibernate-orm#11361

how about something like ^
we tell to run the checks (and if someone doesn't CI should catch it, right?) and we give the links to the IDE pages, but we don't say that they have the styles .... 🫣 🙂

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.

2 participants