[GitHub] [maven-build-cache-extension] AlexanderAshitkin commented on pull request #35: [MBUILDCACHE-33] Allow to set remote cache credentials from env vars

2023-02-06 Thread via GitHub
AlexanderAshitkin commented on PR #35: URL: https://github.com/apache/maven-build-cache-extension/pull/35#issuecomment-1419160683 @gnodet @michael-o I feel like there is no consensus for this change yet, I'm declining this pr so far. Need to agree on the design first. -- This is an

[GitHub] [maven-build-cache-extension] AlexanderAshitkin commented on pull request #35: [MBUILDCACHE-33] Allow to set remote cache credentials from env vars

2023-02-06 Thread via GitHub
AlexanderAshitkin commented on PR #35: URL: https://github.com/apache/maven-build-cache-extension/pull/35#issuecomment-1419159403 > @AlexanderAshitkin have you tried using environment variable interpolation from `settings.xml` ? > Also it would be nice to add a bit of documentation

[GitHub] [maven-build-cache-extension] AlexanderAshitkin commented on pull request #35: [MBUILDCACHE-33] Allow to set remote cache credentials from env vars

2022-11-04 Thread GitBox
AlexanderAshitkin commented on PR #35: URL: https://github.com/apache/maven-build-cache-extension/pull/35#issuecomment-1303960662 > I have expressed my objections in the JIRA issue and why I consider this wrong. Discussed in Jira. There are strong reasons why using credentials from