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

Make the project stucture compatible with JitPack #537

Merged
merged 46 commits into from
Aug 1, 2022
Merged

Make the project stucture compatible with JitPack #537

merged 46 commits into from
Aug 1, 2022

Conversation

CodingDepot
Copy link
Collaborator

This PR allows JitPack to build codyze again

main changes

  • included v2 and v3 as projects instead of just as builds
  • moved jitpack.yml from subproject to root
  • fixed path of gradle task dependencies in codyze-v2
  • changed JitPack jdk from openjdk to eclipse-temurin

side changes

  • applied spotless to the files in codyze-v2 while building

CodingDepot and others added 30 commits December 9, 2021 10:44
@CodingDepot
writing (manually set) version to manifest file
ab513c1
@CodingDepot
first attempt at reading version from manifest file (not working curr…
20c2b69 
…ently)
@CodingDepot
added Name of Implementation
ee54621
@CodingDepot
added VersionProvider (still unsure how to locate manifest file)
c9c69f2
@CodingDepot
removed unnecessary imports
3b5daaf
@CodingDepot
added manifest attributes
efd91be
@CodingDepot
removed unnecessary manifest attributes (already handled by script)
7137070
@CodingDepot
fixed unnoticed typo in field checking
0c14b3f
@CodingDepot
Merge branch 'main' into rh/versionProvider
c829e3e 
# Conflicts:
#	src/main/java/de/fraunhofer/aisec/codyze/Main.java
@CodingDepot
Merge branch 'main' into rh/versionProvider
5874bcb
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
a2bd8f9 
# Conflicts:
#	src/main/java/de/fraunhofer/aisec/codyze/Main.java
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
fd397ba
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
d8b4786
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
d6bd953
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
bb3c7c1
@CodingDepot
Merge branch 'main' of github.com:Fraunhofer-AISEC/codyze into main
fb8b870
@CodingDepot
Merge branch 'main' of github.com:Fraunhofer-AISEC/codyze into main
72ff22b
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
eae2120
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
2327a3f
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
8db6d7b
@CodingDepot
Merge remote-tracking branch 'origin/main' into main
4898cde
@CodingDepot
added gradle build command for composite that delegates to codyze-v2
18f729e
@CodingDepot
spotless changes (building side-effect)
4b01b13
@CodingDepot
removed top-level build.gradle to replace with a jitpack.yml file
a733021
@CodingDepot
added custom install command to jitpack.yml calling codyze-v2 install
b3f0162
@CodingDepot
added wrapper task and commented out failing tasks
75b3e88
@CodingDepot
include codyze versions as projects instead of just the builds
f41bf6b
@CodingDepot
newly generated wrapper for codyze-v2
aa1b07f
@CodingDepot
remove jitpack.yml in subproject
fc5f3a9
@CodingDepot
removed wrapper task again since not needed anymore
5bb83de
@CodingDepot
Copy link
Collaborator Author

Changes in codyze-v2/src/ are caused by spotless, the main changes are everything else.

Maximilian Galanis added 3 commits July 25, 2022 12:18
revert back to multi-build
feae50f
configure jitpack to work with the composite build
846d2f4
Update jitpack.yml
0aa396d 
Perform sdk update before installing java. Should get rid of update warning
@sonarcloud
Copy link

sonarcloud bot commented Aug 1, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

74.4% 74.4% Coverage
0.0% 0.0% Duplication

@fwendland fwendland merged commit b4df5f3 into main Aug 1, 2022
@fwendland fwendland deleted the rh/jitpack-redirect branch August 1, 2022 11:55
fwendland added a commit that referenced this pull request Apr 15, 2024
@CodingDepot @fwendland
Make the project stucture compatible with JitPack (#537)
ff55943 
Co-authored-by: Maximilian Galanis <maximilian.galanis@aisec.fraunhofer.de>
Co-authored-by: Florian Wendland <florian.wendland@aisec.fraunhofer.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fwendland fwendland fwendland approved these changes

@maximilian-galanis maximilian-galanis Awaiting requested review from maximilian-galanis

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@CodingDepot @fwendland