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

For consistency, convert "JETTY_HOME" attribute to "jetty-home" #9152

Conversation

gregpoulos
Copy link
Contributor

This is the implementation of a suggestion from @sbordet.

Greg Poulos added 2 commits January 10, 2023 15:01
… into jetty-12.0.x-documentation-lowercase-jetty_home-attribute

* 'jetty-12.0.x' of https://github.com/eclipse/jetty.project:
  Issue jetty#9167 - making assumption in flaky test
  jetty 12.0.x cleanup duplicate osgi pom metadata (jetty#9093)
  Jetty 12 - Add tests in util/resource for alternate FileSystem implementations (jetty#9149)
  Cleanup non-retainable `Retainable`s (jetty#9159)
  Fixes retainability of special Chunks (jetty#9073)
  TCK: Dispatch forward and includes attributes do not meet the spec (jetty#9074)
  re-enable h3 tests (jetty#8773)
  More fundamental test case
  Reorganization of jetty-client classes. (jetty#9127)
  Removing @disabled from SslUploadTest
  Removing @disabled from jetty-start
  Bump maven.version from 3.8.6 to 3.8.7
  Bump maven.version from 3.8.6 to 3.8.7
@sbordet sbordet self-requested a review January 16, 2023 21:42
Copy link
Contributor

@sbordet sbordet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@sbordet sbordet merged commit e9f3115 into jetty:jetty-12.0.x Jan 17, 2023
gregpoulos pushed a commit to gregpoulos/jetty.project that referenced this pull request Jan 17, 2023
…x-document-modules

* upstream/jetty-12.0.x:
  Documentation: for consistency, convert "JETTY_HOME" attribute to "jetty-home" (jetty#9152)
  suppress test stacks
  Jetty 12 temp directory cleanup (jetty#9153)
  Only use GraalIssue5720PathResourceFactory when truly needed
  Refactor Graal native-image resource: handling
  Move GraalVM Native-Image code to NativeImagePathResource/-Factory
  PathResource: Fix "isAlias" issue with resource: URIs
  PathResource: Work-around false positive CodeQL warning
  PathResourceFactory: Ignore certain RuntimeExceptions upon init
  PathResource: Selectively enable resource: for GraalVM Native Image
  URIUtil: Removed unused code
  URIUtil: Suppress CodeQL false positive error about path injection
  util: Add support for GraalVM Native-Image resource:-URIs and Paths
gregpoulos pushed a commit to gregpoulos/jetty.project that referenced this pull request Jan 17, 2023
…oject into jetty-12.0.x-documentation-operations-logging

* 'jetty-12.0.x' of https://github.com/gregpoulos/jetty.project:
  Documentation: for consistency, convert "JETTY_HOME" attribute to "jetty-home" (jetty#9152)
  suppress test stacks
  Jetty 12 temp directory cleanup (jetty#9153)
  Only use GraalIssue5720PathResourceFactory when truly needed
  Refactor Graal native-image resource: handling
  Move GraalVM Native-Image code to NativeImagePathResource/-Factory
  PathResource: Fix "isAlias" issue with resource: URIs
  PathResource: Work-around false positive CodeQL warning
  PathResourceFactory: Ignore certain RuntimeExceptions upon init
  PathResource: Selectively enable resource: for GraalVM Native Image
  URIUtil: Removed unused code
  URIUtil: Suppress CodeQL false positive error about path injection
  util: Add support for GraalVM Native-Image resource:-URIs and Paths
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