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

Upgrade that working on MacOS #590

Merged
merged 3 commits into from
Apr 10, 2023
Merged

Upgrade that working on MacOS #590

merged 3 commits into from
Apr 10, 2023

Conversation

alyxdow
Copy link
Contributor

@alyxdow alyxdow commented Apr 9, 2023

Explanation:

  • The first line #!/bin/bash specifies the shell to be used as bash.
  • The if statement checks the output of the git status command. The grep command searches for any lines that start with a non-space character (not counting the ?? status, which indicates untracked files), indicating that there are changes that need to be committed.
  • git pull pulls the latest changes from the remote repository.
  • source venv/bin/activate activates the virtual environment.
  • pip install --use-pep517 --upgrade -r requirements.txt upgrades the required packages in the activated virtual environment.

bmaltais and others added 3 commits April 7, 2023 08:20
Explanation:

* The first line #!/bin/bash specifies the shell to be used as bash.
* The if statement checks the output of the git status command. The grep command searches for any lines that start with a non-space character (not counting the ?? status, which indicates untracked files), indicating that there are changes that need to be committed.
* git pull pulls the latest changes from the remote repository.
* source venv/bin/activate activates the virtual environment.
* pip install --use-pep517 --upgrade -r requirements.txt upgrades the required packages in the activated virtual environment.
@jstayco
Copy link
Contributor

jstayco commented Apr 10, 2023

@bernardmaltais This is a decent stopgap solution until I have my installer/launcher branch ready to go. Upgrade scripts will be obsolete after though (it's now a switch on the launcher).

@bmaltais bmaltais changed the base branch from master to dev April 10, 2023 22:49
@bmaltais
Copy link
Owner

Thank you. I will merge knowing it will be replaced eventually...

@bmaltais bmaltais merged commit 99503e3 into bmaltais:dev Apr 10, 2023
@alyxdow alyxdow deleted the patch-1 branch April 11, 2023 03:59
bmaltais pushed a commit that referenced this pull request Jun 15, 2023
prodigyopt, arbitrary dataset etc.
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