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

Bump to Logback 1.3.0-beta0 #832

Merged
merged 2 commits into from
Aug 14, 2022
Merged

Bump to Logback 1.3.0-beta0 #832

merged 2 commits into from
Aug 14, 2022

Conversation

brenuart
Copy link
Collaborator

  • Build against Logback 1.3.0
  • Execute backward compatibility tests against latest Logback 1.2.x
  • Comment links to Logback javadoc (generation fails under Java8 because logback docs have replace "package-list" by "element-list")

Build against Logback 1.3.0
Execute backward compatibility tests against latest Logback 1.2.x
Javadoc generation fails under Java8 because the Logback javadoc does not contain a "package-list" index file anymore since version 1.3.0. It seems to work however with Java 9 and 11.

Note: "package-list" has been replaced by "element-list" in Java 9... Newer jdk versions seem to look for "element-list" first and fallback to "package-list".

A possible workaround is to build the project (and generate javadoc) using Java 9 instead of 8. In the meantime links to Logback apidoc are commented.
@brenuart brenuart added this to the 7.3 milestone Aug 14, 2022
@codecov
Copy link

codecov bot commented Aug 14, 2022

Codecov Report

Merging #832 (8ced2d8) into main (38cf6ff) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #832   +/-   ##
=========================================
  Coverage     70.97%   70.97%           
  Complexity     1255     1255           
=========================================
  Files           167      167           
  Lines          4807     4807           
  Branches        498      498           
=========================================
  Hits           3412     3412           
  Misses         1150     1150           
  Partials        245      245           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@brenuart brenuart merged commit d7e48a2 into main Aug 14, 2022
@brenuart brenuart deleted the logback13 branch August 14, 2022 15:39
@philsttr philsttr added the type/dependencies Pull requests that update a dependency label Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/dependencies Pull requests that update a dependency
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants