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.

Sign up for GitHub

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

Jump to bottom

Tinylog #8226

Merged
merged 23 commits into from
Dec 18, 2021
Merged

Tinylog #8226

merged 23 commits into from
Dec 18, 2021

Conversation

koppor
Copy link
Member

@koppor koppor commented Nov 8, 2021 •
edited by Siedlerchr
Loading

Fixes #8007
Fixes #5475

Debug log level is only possible with java property -Dtinylog.level=debug
Tiny log freezes the config after loading

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

@Siedlerchr
test around with getting rid of log4j
247c2a0
@Siedlerchr
advanced message formatter
4dac2a2
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into tinylog
91c1afb 
* upstream/main: (259 commits)
  Fix exception on preview style edit and selection (#8293)
  Fix icon picker excpetion in groups dialog (#8290)
  Add doc tests (#8242)
  fix merge conflict
  Squashed 'buildres/csl/csl-locales/' changes from 0cc3885f61..d5ee85de8e
  Squashed 'buildres/csl/csl-styles/' changes from 3a6a0a7..3bb4b5f
  Consider directory pattern when checking if a file can be moved (#8244)
  Bump byte-buddy-parent from 1.12.1 to 1.12.2 (#8285)
  Bump unirest-java from 3.13.3 to 3.13.4 (#8283)
  Bump checkstyle from 9.1 to 9.2 (#8284)
  Bump classgraph from 4.8.135 to 4.8.137 (#8282)
  Do not resize main table columns in search dialog window (#8253)
  Fix NegativeArraySizeException (#8270)
  Update deployment.yml
  TEst
  Grand unified library properties (GRULPS) (#8264)
  Fixes RFC fetcher test case (#8271)
  Update Gradle Wrapper from 7.2 to 7.3 (#8246)
  Observable Preferences H (WorkingDir, getUser, NameFormatter, Version, SpecialFields) (#8260)
  Bump checkstyle from 9.0.1 to 9.1 (#8266)
  ...
@Siedlerchr
fix compile errors and update tinylog
c061b88
@Siedlerchr
Copy link
Member

Siedlerchr commented Dec 4, 2021 •
edited
Loading

Issue resolved tinylog-org/tinylog#223

@Siedlerchr
Merge remote-tracking branch 'upstream/main' into tinylog
6fc081a 
* upstream/main:
  Fix xmp import (#8311)
  New Crowdin updates (#8308)
  Fix file-filter in XMP Export (#8307)
  New Crowdin updates (#8306)
  Bump h2-mvstore from 1.4.200 to 2.0.202 (#8302)
  Remove bytebuddy (#8300)
  Bump postgresql from 42.2.24 to 42.3.1 (#8301)
  Update Gradle Wrapper from 7.3 to 7.3.1. (#8299)
  Apply search string to open library when keep search string is selected (#8294)
@Siedlerchr
update to Release version
13d4965
@Siedlerchr
checkstyle
39188dd
@Siedlerchr
remove log4j stuff from jlink
bb778e3
@Siedlerchr Siedlerchr mentioned this pull request Dec 10, 2021
Closed
@Siedlerchr
update to 2.4.1
2832c1d
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into tinylog
d86bcfe 
* upstream/main:
  New translations JabRef_en.properties (Turkish) (#8321)
  snap: Remove superfluous files post build (#8319)
  Fix IEEE test (#8320)
  New translations JabRef_en.properties (German) (#8318)
  Fixed exception handling (#8314)
  New Crowdin updates (#8312)
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into tinylog
cc924e0 
* upstream/main:
  Don't register any database changes to the indexer while dropping a file (#8334)
  Fix ACM fetcher (#8338)
  Squashed 'buildres/csl/csl-styles/' changes from 3bb4b5f..60bf7d5
  Exception shouldn't happen when pasting an entry with a publication date-range of form yyyy-yyyy (#8247)
  Refactor Sidepane logic (#8202)
  New translations JabRef_en.properties (Japanese) (#8331)
  Bump bcprov-jdk15on from 1.69 to 1.70 (#8333)
  Update Controlsfx to 11.1.1 (#8330)
  Update citeproc (#8329)
  Bump classgraph from 4.8.137 to 4.8.138 (#8327)
  Bump junit-platform-launcher from 1.8.1 to 1.8.2 (#8326)
  Remove outdated gradle deps check (#8324)
  Bump junit-jupiter from 5.8.1 to 5.8.2 (#8325)
  Bump libreoffice from 7.2.2 to 7.2.3 (#8328)
  Remove outdated ignores
@Siedlerchr
Implement tinylog custom writers
3c4764a
@Siedlerchr Siedlerchr marked this pull request as ready for review December 17, 2021 22:52
@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Dec 18, 2021
@Siedlerchr Siedlerchr mentioned this pull request Dec 18, 2021
Open
tobiasdiez
tobiasdiez requested changes Dec 18, 2021
View reviewed changes
Copy link
Member

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

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

Nice!

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
src/main/java/module-info.java Outdated Show resolved Hide resolved
Siedlerchr and others added 4 commits December 18, 2021 19:29
@Siedlerchr @tobiasdiez
Update CHANGELOG.md
8978f76 
Co-authored-by: Tobias Diez <tobiasdiez@gmx.de>
@Siedlerchr @tobiasdiez
Update CHANGELOG.md
b48c0b5 
Co-authored-by: Tobias Diez <tobiasdiez@gmx.de>
@Siedlerchr
sort module info
8703a0e
@Siedlerchr
remove commented out line
8e5e94a 
no name avaiable
k3KAW8Pnf7mkmdSMPHz27
k3KAW8Pnf7mkmdSMPHz27 approved these changes Dec 18, 2021
View reviewed changes
Copy link
Sponsor Member

@k3KAW8Pnf7mkmdSMPHz27 k3KAW8Pnf7mkmdSMPHz27 left a comment

Choose a reason for hiding this comment

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

LGTM

@k3KAW8Pnf7mkmdSMPHz27 k3KAW8Pnf7mkmdSMPHz27 merged commit 1c7da01 into main Dec 18, 2021
This was referenced Dec 18, 2021
Closed
Closed
Siedlerchr added a commit that referenced this pull request Dec 20, 2021
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into jabrefonline
2762345 
* upstream/main: (46 commits)
  New Crowdin updates (#8349)
  Bump pdfbox from 2.0.24 to 2.0.25 (#8345)
  Bump fontbox from 2.0.24 to 2.0.25 (#8348)
  Bump xmlunit-core from 2.8.3 to 2.8.4 (#8347)
  Bump tika-core from 2.1.0 to 2.2.0 (#8346)
  Added missing executable bindings to various commands (#8342)
  Update Gradle Wrapper from 7.3.1 to 7.3.2. (#8343)
  Fix issues with writing metadata to pdfs (#8332)
  add tinylog test (#8339)
  Tinylog (#8226)
  Don't register any database changes to the indexer while dropping a file (#8334)
  Fix ACM fetcher (#8338)
  Squashed 'buildres/csl/csl-styles/' changes from 3bb4b5f..60bf7d5
  Exception shouldn't happen when pasting an entry with a publication date-range of form yyyy-yyyy (#8247)
  Refactor Sidepane logic (#8202)
  New translations JabRef_en.properties (Japanese) (#8331)
  Bump bcprov-jdk15on from 1.69 to 1.70 (#8333)
  Update Controlsfx to 11.1.1 (#8330)
  Update citeproc (#8329)
  Bump classgraph from 4.8.137 to 4.8.138 (#8327)
  ...

# Conflicts:
#	build.gradle
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tobiasdiez tobiasdiez tobiasdiez requested changes

@k3KAW8Pnf7mkmdSMPHz27 k3KAW8Pnf7mkmdSMPHz27 k3KAW8Pnf7mkmdSMPHz27 approved these changes

Assignees
No one assigned
Labels
Logging status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers
Projects
None yet
Milestone
No milestone
5 participants
@koppor @Siedlerchr @k3KAW8Pnf7mkmdSMPHz27 @tobiasdiez @ThiloteE