Skip to content
This repository was archived by the owner on Sep 29, 2023. It is now read-only.

app name nativefier-postFixHash#888

Closed
0xnu wants to merge 4 commits into
nativefier:masterfrom
0xnu:master
Closed

app name nativefier-postFixHash#888
0xnu wants to merge 4 commits into
nativefier:masterfrom
0xnu:master

Conversation

@0xnu

@0xnu 0xnu commented Dec 26, 2019

Copy link
Copy Markdown

For issue #851

  • Removed the nativefier-postFixHash appended to the generated app name.

Signed-off-by: Finbarrs Oketunji <oketunjifinbarrs@gmail.com>
@mark-carlson

Copy link
Copy Markdown

I opened a PR on this one too. #892
Thanks!

Signed-off-by: Finbarrs Oketunji <oketunjifinbarrs@gmail.com>

@mark-carlson mark-carlson left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

Comment thread src/build/buildApp.js
Signed-off-by: Finbarrs Oketunji <oketunjifinbarrs@gmail.com>
Comment thread src/build/buildApp.js
@TheCleric

Copy link
Copy Markdown
Collaborator

@0xnu I think we may have gotten this resolved since you submitted the PR. Can you verify, and if you still feel like it's worth pursuing, get master merged into this branch so we can take a look again?

@TheCleric

Copy link
Copy Markdown
Collaborator

Closing this as the issue it references no longer seems to be a problem, and as well this branch would need cleanup.

@0xnu if you still see a use case for the code you have here, please get the conflicts resolved and let us know what the PR would achieve at this point. We'd still be happy to take a look at it. 😄

@TheCleric TheCleric closed this May 10, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants