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

ref: Rename DEV_MODE as CONTINUE_ANYWAY #434

Merged
merged 7 commits into from
Apr 1, 2024
Merged

Conversation

RobertGemmaJr
Copy link
Member

@RobertGemmaJr RobertGemmaJr commented Mar 20, 2024

The DEV_MODE flag is only true when the user elects "continue anyway" (e.g. for testing an experiment without the hardware explicitly connected. The rename to CONTINUE_ANYWAY better indicates the state of the application though

  • Renames DEV_MODE as CONTINUE_URL
  • Pulls process.env.ELECTRON_START_URL into its own variable

@RobertGemmaJr RobertGemmaJr self-assigned this Mar 20, 2024
@RobertGemmaJr RobertGemmaJr changed the base branch from main to v3.3.1 March 20, 2024 19:15
Copy link

github-actions bot commented Mar 20, 2024

Visit the preview URL for this PR (updated for commit 472eee2):

https://ccv-honeycomb--pr434-ref-continue-anyway-n9ak36uq.web.app

(expires Tue, 02 Apr 2024 18:57:05 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4ace1dcea913a952d2a1af84b94a4421bf36e610

@RobertGemmaJr RobertGemmaJr added electron Organization: Issue in regards Electron code 3.3 Versioning: Issue in regards to version 3.3 release labels Mar 20, 2024
@RobertGemmaJr RobertGemmaJr linked an issue Mar 20, 2024 that may be closed by this pull request
@RobertGemmaJr RobertGemmaJr merged commit 8b14447 into v3.3.1 Apr 1, 2024
8 checks passed
@RobertGemmaJr RobertGemmaJr deleted the ref-continue-anyway branch April 1, 2024 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.3 Versioning: Issue in regards to version 3.3 release electron Organization: Issue in regards Electron code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename DEV_MODE as CONTINUE_ANYWAY
2 participants