Skip to content

Conversation

ckittl
Copy link
Member

@ckittl ckittl commented Jan 7, 2022

Resolves #125

Is currently block by failing CI pipeline (and thus missing snapshot deployment) in PowerSystemUtils (ie3-institute/PowerSystemUtils#205 and ie3-institute/PowerSystemUtils#207)

@ckittl ckittl self-assigned this Jan 7, 2022
@ckittl ckittl added the dependencies Pull requests that update a dependency file label Jan 7, 2022
@sonarqubegithubprchecks

This comment has been minimized.

@sonarqubegithubprchecks

This comment has been minimized.

@codecov
Copy link

codecov bot commented Jan 7, 2022

Codecov Report

Merging #126 (61f5a67) into dev (6b1605b) will not change coverage.
The diff coverage is n/a.

❗ Current head 61f5a67 differs from pull request most recent head 0ffa84b. Consider uploading reports for the commit 0ffa84b to get more accurate results
Impacted file tree graph

@@          Coverage Diff          @@
##             dev    #126   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files         25      25           
  Lines       1512    1512           
  Branches     236     236           
=====================================
  Misses      1512    1512           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d31b35b...0ffa84b. Read the comment docs.

@sonarqubegithubprchecks

This comment has been minimized.

johanneshiry
johanneshiry previously approved these changes Jan 11, 2022
@sonarqubegithubprchecks

This comment has been minimized.

Comment on lines +1 to +6
# Workaround for splotless with java 17
org.gradle.jvmargs=--add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \
--add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED
Copy link
Member Author

Choose a reason for hiding this comment

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

This is only a workaround and shall be fixed with #130

@sonarqubegithubprchecks

This comment has been minimized.

1 similar comment
@sonarqubegithubprchecks
Copy link

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (0.00% Estimated after merge)
  • No duplication informationNo duplication information (0.00% Estimated after merge)

Project ID: edu.ie3:OSMoGrid

View in SonarQube

@ckittl
Copy link
Member Author

ckittl commented Jan 17, 2022

Also removes the deprecated Java implementation. It can be found in this archive branch.

@ckittl ckittl marked this pull request as ready for review January 17, 2022 15:28
@ckittl ckittl requested review from a team and johanneshiry January 17, 2022 15:28
@t-ober t-ober merged commit c0aad4b into dev Jan 18, 2022
@t-ober t-ober deleted the ck/#125-snapshotDependencies branch January 18, 2022 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blockedExternally dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rely on snapshot dependencies

3 participants