Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use profile for overriding properties when building on the CI #11232

Merged
merged 1 commit into from
May 7, 2021
Merged

Conversation

normanmaurer
Copy link
Member

Motivation:

0f25213 introduced some properties that were used to make builds more stable on the ci. All of these properties were duplicated everywhere, this made it hard to maintain

Modifications:

  • Add profile which sets the properties.
  • Just use the profile when build on the ci

Result:

Easier to maintain custom properties for the ci build

Motivation:

0f25213 introduced some properties that were used to make builds more stable on the ci. All of these properties were duplicated everywhere, this made it hard to maintain

Modifications:

- Add profile which sets the properties.
- Just use the profile when build on the ci

Result:

Easier to maintain custom properties for the ci build
@normanmaurer normanmaurer added this to the 4.1.64.Final milestone May 7, 2021
@normanmaurer normanmaurer merged commit 8676203 into 4.1 May 7, 2021
@normanmaurer normanmaurer deleted the ci branch May 7, 2021 16:24
normanmaurer added a commit that referenced this pull request May 7, 2021
Motivation:

0f25213 introduced some properties that were used to make builds more stable on the ci. All of these properties were duplicated everywhere, this made it hard to maintain

Modifications:

- Add profile which sets the properties.
- Just use the profile when build on the ci

Result:

Easier to maintain custom properties for the ci build
raidyue pushed a commit to raidyue/netty that referenced this pull request Jul 8, 2022
…11232)

Motivation:

0f25213 introduced some properties that were used to make builds more stable on the ci. All of these properties were duplicated everywhere, this made it hard to maintain

Modifications:

- Add profile which sets the properties.
- Just use the profile when build on the ci

Result:

Easier to maintain custom properties for the ci build
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.

None yet

2 participants