Skip to content

[MRESOLVER-302] Addendum#360

Merged
cstamas merged 17 commits intoapache:masterfrom
cstamas:MRESOLVER-302-addendum
Nov 9, 2023
Merged

[MRESOLVER-302] Addendum#360
cstamas merged 17 commits intoapache:masterfrom
cstamas:MRESOLVER-302-addendum

Conversation

@cstamas
Copy link
Copy Markdown
Member

@cstamas cstamas commented Nov 9, 2023

Changes:

  • rename closeable session to lessen repetition in source
  • defer creation of LRM to ctor of DefaultCloseableSession
  • support chained LRM out of the box
  • make AuthenticationContext not need session (as it does not use it directly)

https://issues.apache.org/jira/browse/MRESOLVER-302

Changes:
* rename closeable session to lessen repetition in source
* make builder implement session interface but explain what is the intent here

---

https://issues.apache.org/jira/browse/MRESOLVER-302
@cstamas cstamas self-assigned this Nov 9, 2023
@cstamas cstamas marked this pull request as ready for review November 9, 2023 14:37
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
Comment thread maven-resolver-api/src/main/java/org/eclipse/aether/RepositorySystemSession.java Outdated
@cstamas cstamas requested a review from gnodet November 9, 2023 23:08
@cstamas cstamas merged commit 43cae03 into apache:master Nov 9, 2023
@cstamas cstamas deleted the MRESOLVER-302-addendum branch November 9, 2023 23:49
@jira-importer
Copy link
Copy Markdown

Resolve #974

1 similar comment
@jira-importer
Copy link
Copy Markdown

Resolve #974

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.

3 participants