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

added conveyor packaging for fx-sampler #580

Merged
merged 14 commits into from
Sep 26, 2023
Merged

Conversation

ennerf
Copy link
Collaborator

@ennerf ennerf commented Apr 6, 2023

I added a basic Conveyor configuration for the sampler app and uploaded a signed test version to chartfx-sampler. I also added a preliminary square icon to match the macOS guidelines, and fixed a few issues that broke the sampler app when running from jar files (e.g. resource paths with . and .. don't work)

It probably doesn't make sense to get signing certificates for a sampler project. Maybe Hydraulic (ping @mikehearn) can handle the signing/hosting like they do for some other projects?

I'm leaving it as a draft PR for now because there is some issue with the installed versions. See conveyor#56

Update
The issue occurs when executing with bin as the working directory. Executing it manually from somewhere else works. Absolute vs relative paths do not make a difference. My best guess is that it's a bug in fx-sampler.

@ennerf ennerf temporarily deployed to coverage April 6, 2023 14:44 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 6, 2023 14:44 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 6, 2023 14:44 — with GitHub Actions Inactive
@codecov
Copy link

codecov bot commented Apr 6, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (e774d7b) 48.06% compared to head (d4b5a66) 48.05%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #580      +/-   ##
============================================
- Coverage     48.06%   48.05%   -0.02%     
+ Complexity     6198     6197       -1     
============================================
  Files           374      374              
  Lines         38276    38278       +2     
  Branches       6102     6103       +1     
============================================
- Hits          18398    18395       -3     
- Misses        18734    18737       +3     
- Partials       1144     1146       +2     
Files Coverage Δ
...fair_acc/chartfx/utils/SimplePerformanceMeter.java 87.69% <0.00%> (-1.38%) ⬇️

... and 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ennerf ennerf temporarily deployed to coverage April 6, 2023 15:38 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 6, 2023 15:38 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 6, 2023 15:38 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 14:15 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 14:15 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 14:15 — with GitHub Actions Inactive
@ennerf ennerf marked this pull request as ready for review April 7, 2023 14:26
@ennerf
Copy link
Collaborator Author

ennerf commented Apr 7, 2023

The FXSampler project has some issues w/ the Conveyor packaging format, but I managed to work around it. The packaging should be fully working now.

Test release: https://files.hebi.us/internal/demo/chartfx-sampler/download.html

@ennerf ennerf temporarily deployed to coverage April 7, 2023 15:26 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 15:26 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 15:26 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 16:01 — with GitHub Actions Inactive
@ennerf ennerf temporarily deployed to coverage April 7, 2023 16:01 — with GitHub Actions Inactive
@sonarcloud
Copy link

sonarcloud bot commented Jul 17, 2023

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 1 Code Smell

No Coverage information No Coverage information
1.6% 1.6% Duplication

(cherry picked from commit bc18a61)
(cherry picked from commit c52c44f)
(cherry picked from commit d983471)
(cherry picked from commit e9e2f6b)
(cherry picked from commit e1c2538)
(cherry picked from commit ca0689b)
(cherry picked from commit d1cf767)
(cherry picked from commit 74ec30a)
@sonarcloud
Copy link

sonarcloud bot commented Sep 26, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

No Coverage information No Coverage information
3.0% 3.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

Copy link
Member

@wirew0rm wirew0rm left a comment

Choose a reason for hiding this comment

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

👍

@wirew0rm wirew0rm merged commit 08a8be4 into fair-acc:main Sep 26, 2023
4 of 8 checks passed
@ennerf ennerf deleted the conveyor branch September 26, 2023 11:12
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.

3 participants