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

fix(Java/.NET): version by properties file #259

Closed
wants to merge 4 commits into from

Conversation

texastony
Copy link
Contributor

@texastony texastony commented Mar 30, 2024

Issue #, if available:
Centralize all versioning variables.

Description of changes:

Centralize all versioning variables in the root properties file.
Parse that properties file in .NET and Java project files.
This should simplify our publishing process,
particularly for test/development pushes to internal CodeArtifact
to run integration tests against the DB-ESDK or ESDK.

Squash/merge commit message, if applicable:

fix(Java/.NET): version by properties file

TODO:

  • Update .releaserc.cjs, removing AssemblyInfo.cs references
  • Confirm we do not need *.sln or AssemblyInfo.cs files.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@texastony texastony force-pushed the tony/fix-version-by-properties branch from eab04f0 to 55b875c Compare March 30, 2024 19:33
@texastony texastony marked this pull request as ready for review March 30, 2024 19:34
@texastony texastony requested a review from a team as a code owner March 30, 2024 19:34
@texastony texastony force-pushed the tony/fix-version-by-properties branch from 43e7ee9 to 2c3598d Compare May 20, 2024 21:10
@texastony texastony force-pushed the tony/fix-version-by-properties branch from 2c3598d to 9b5e748 Compare May 20, 2024 21:14
@texastony texastony closed this Sep 13, 2024
@texastony texastony deleted the tony/fix-version-by-properties branch September 13, 2024 19:09
ajewellamz pushed a commit that referenced this pull request Dec 18, 2024
Reading environment variables MUST occur before
registering any tasks that use them.
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