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

feature: add macos signing + notarization. #1191

Merged
merged 1 commit into from
Nov 1, 2024
Merged

Conversation

wkelly17
Copy link
Contributor

@wkelly17 wkelly17 commented Oct 29, 2024

feature:

  • add macos signing + notarization.
  • Upgrade install4j version…
  • Change secrets management

This change is Reviewable

Copy link
Collaborator

@jsarabia jsarabia left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: 0 of 3 files reviewed, all discussions resolved (waiting on @danparisd and @PurpleGuitar)

Copy link
Collaborator

@danparisd danparisd left a comment

Choose a reason for hiding this comment

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

LGTM

@danparisd
Copy link
Collaborator

@jsarabia You can merge this and make a release at your discretion!

@jsarabia jsarabia merged commit f4cd0f7 into dev Nov 1, 2024
9 of 10 checks passed
@jsarabia jsarabia deleted the wk-testMacSigning branch November 1, 2024 16:22
wkelly17 added a commit that referenced this pull request Nov 8, 2024
jsarabia added a commit that referenced this pull request Nov 8, 2024
* find error that was with previous build

* Fixed bug where verse marker was not added after editing chapter. (#1190)

The issue here is that we are checking based on the label and not the formatted label. The label for chapter maker 2 and verse marker 2 is just "2", however, the formatted label is "chapter-marker-2" and "verse-marker-2"

* Protect setting recordedVerses (#1192)

* Protect setting recordedVerses

A crash seems to be connected to a concurrent modification issue

Also clean up in undock

* Update AudioWorkspaceView.kt

* feature: add macos signing + notarization.  Upgrade install4j version. Change secretes management in build (#1191)

* Update VERSION

* finish rebase dev adn debug signing

---------

Co-authored-by: Darrell Cole Hill <[email protected]>
Co-authored-by: Joe <[email protected]>
wkelly17 added a commit that referenced this pull request Jan 2, 2025
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