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

1.3.0 #288

Merged
merged 13 commits into from
Jul 5, 2024
Merged

1.3.0 #288

merged 13 commits into from
Jul 5, 2024

Conversation

Nusiq
Copy link
Member

@Nusiq Nusiq commented Jul 5, 2024

  • Added --update flag to regolith install-all
  • Fixed application crashes when running regolith watch on projects without RP or BP
  • During filter installation the copying of the data folder is now executed before the installation of the dependencies.

Nusiq and others added 13 commits March 3, 2024 12:59
…Errorf(...) instead of WrapErrorf(err, ...)).
…ode:

- The regolith watch and regolith run only share the setup, but run from separate functions.
- The file observers are created only if the path in the config is not an empty string.
Fix `regolith watch` command for projects that only use RP or BP (but not both)
…s used only to control whether or not to show an optional warning message.
The install function used to take an 'add' argument to determine whether the filter should be added
to specified profiles or not. The list of profiles was used only when the add flag was set to true.
If user didn't provide the profiles explicitly it was set to a list with a single item - 'default'.
Now, the internal behavior is changed - there is no 'add' flag, but the 'default' profile is
added to the list only if the user uses the --profiles flag in the command. The behavior of the
program from the user point of view is the same. Internally the Install funciton expects
an empty list if no profiles need to be updated.
Add --update flag for install-all subcommand
Old: '{: .notice--warning}'
New: ':::: warning'
Move copying data before installing dependencies
@Nusiq Nusiq requested a review from stirante July 5, 2024 19:26
@stirante stirante merged commit ed8bb81 into main Jul 5, 2024
2 checks passed
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.

2 participants