Skip to content

[MRESOLVER-402] Fix property handling#29

Merged
cstamas merged 2 commits intomasterfrom
fix-property-handling
Sep 7, 2023
Merged

[MRESOLVER-402] Fix property handling#29
cstamas merged 2 commits intomasterfrom
fix-property-handling

Conversation

@cstamas
Copy link
Copy Markdown
Member

@cstamas cstamas commented Sep 7, 2023

Properties are laid down as (former may be overridden by latter):

  • internally mandatory ones (UA, startTime)
  • Java System Properties
  • Ant project properties (all Ant project properties that has all)

Fixes issue where Java System Properties were not present in Resolver config properties (Resolver strictly observes config properties only), and added missing one.

Properties are laid down as:
* internally mandatory ones (UA, startTime)
* Java System Properties
* Ant project properties (all Ant project properties that has all)
@cstamas cstamas requested a review from slachiewicz September 7, 2023 07:10
@cstamas cstamas self-assigned this Sep 7, 2023
This also makes env stuff available.
@cstamas cstamas changed the title Fix property handling (followup to update) [MRESOLVER-402] Fix property handling (followup to update) Sep 7, 2023
@cstamas cstamas changed the title [MRESOLVER-402] Fix property handling (followup to update) [MRESOLVER-402] Fix property handling Sep 7, 2023
@cstamas cstamas merged commit bf5aeca into master Sep 7, 2023
@cstamas cstamas deleted the fix-property-handling branch September 7, 2023 08:09
@jira-importer
Copy link
Copy Markdown

Resolve #96

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