-
Notifications
You must be signed in to change notification settings - Fork 33
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
🔖 Release 3.4.2 #279
Merged
Merged
🔖 Release 3.4.2 #279
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🔀 Backmerge master
* Update azure-pipelines.yml for Azure Pipelines
…de. (#190) * Fixes 'make.exe.exe' typo. Adds disable sentry prompt option for VSCode. * Fix sentry-off flag * change sentry-off to no-sentry
* Fix and move no-sentry to default options. Setup framework for analytics * Fix some things. --toggle-analytics works * Analytics should be working. Need to figure out which commands matter * Make use-analytics a choice not a toggle * Show no-analytics flag feedback so user knows it works * Analytics appear to be working! * Reset uid to None * Compress code a bit * Remove interactive command analytics. Fix info-project typo * Move GA config to cli.pros. Fixes --use-analytics
* Start of more upload options * Adds project icon, name, and description. Use name/description="string" * pros v5 rm-program command * rm program literals * Remove extra print messages. Kernel version = None when no project
🔀 Backmerge master
* Adds kv_read and kv_write functions (UNTESTED) to V5Device class * Add commands to read and set kernel variables * Add descriptions to set_variable and read_variable commands * Finish reading kernel variables. Use None for unknown rx length * Trim kernel variable value and ensure null terminated. * Move rv and sv to v5 commands * Working set command. Maximum lengths based on brain's screen space * get and set aliases
* Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines * Update azure-pipelines.yml for Azure Pipelines
backmerge Release 3.3.2 (#212)
🔀 Backmerge master
* old code (not working) * use correct txrx function * return set value
* Added file redirect click.option to terminal.py * Got rid of indentation error * Added code for output * Redirects standard output stream to file * Outputs to both terminal and file * Modified imported modules * Added line at end * changes on windows * Update pros/cli/terminal.py for terminal file output Co-authored-by: Will Xu <54247087+WillXuCodes@users.noreply.github.com> * Removed ident error Co-authored-by: BennyBot <48661356+BennyBot@users.noreply.github.com> Co-authored-by: Will Xu <xu1321@purdue.edu> Co-authored-by: Benjamin Davis <davi1561@purdue.edu> Co-authored-by: Will Xu <54247087+WillXuCodes@users.noreply.github.com> Co-authored-by: BennyBot <48661356+BennyBot@users.noreply.github.com>
Fix typo in comment
* Move Issue Template Files * Update BUG_REPORT.md
…os-cli into develop"" This reverts commit 0929e41.
…-actions ✨ Pyinstaller GitHub actions
* Run make clean after uninstall temp * Add flag
This reverts commit ea02936.
This reverts commit d4ec163.
* analytics run concurrently with commands * add requests-futures to requirements * remove prints * remove one more print * remove some more prints * remove commented lines
✨No invalid characters in new project directory name
* Added check for valid directory name when creating a project * Removed pyinstaller from requirements.txt * Revert "Removed pyinstaller from requirements.txt" This reverts commit ea02936. * Revert "Added check for valid directory name when creating a project" This reverts commit d4ec163. * Added check for invalid new project directory name * Added check for invalid directory characters * Added check for non-ASCII characters --------- Co-authored-by: Andrew Chen <chen4192@purdue.edu> Co-authored-by: Andrew Chen <andrew.benjamin.chen@gmail.com>
* Initial commit for prompt * Created flag for kernel templates * Added warning code to apply_template * Removed accidental extra lines * Removed accidental whitespace * ✨ Run make clean after uninstall template (#244) * Run make clean after uninstall temp * Add flag * Update Version to 3.4.0 * ✨Added message for terminal connection (#254) * Add beta depot and flag to upgrade * Only use beta depot if specified * Change --pros-4 to --beta * Add ReleaseChannel enum to conductor.py * Fix up some lines * Change the url to reflect json moved in repo * Change name of beta depot * CHange beta depot name too * Fixed making PROS 4 when trying to make PROS 3 project * Change beta enum to bool * WIP fix * Remove debug prints * Add beta_local_templates category * Fix conductor.config issue * Resolve conflict * Display all templates with --beta in pros c q * Add beta flag * Uninstall charset_normalizer * Allow uninstalling invalid templates * Initial commit for prompt * Added warning code to apply_template * Removed accidental extra lines * Removed accidental whitespace * Created flag for kernel templates * Add beta depot and flag to upgrade * Only use beta depot if specified * Change --pros-4 to --beta * Add ReleaseChannel enum to conductor.py * Fix up some lines * Change the url to reflect json moved in repo * Change name of beta depot * CHange beta depot name too * Fixed making PROS 4 when trying to make PROS 3 project * Change beta enum to bool * WIP fix * Fix conductor.config issue * Remove debug prints * Add beta_local_templates category * Display all templates with --beta in pros c q * Add beta flag * Allow uninstalling invalid templates * Change url * Fix url * Temp fix * Update pros/cli/conductor.py Co-authored-by: Kunwar Sahni <kunwar.sahni01@gmail.com> * Initial commit for prompt * Added warning code to apply_template * Removed accidental extra lines * Removed accidental whitespace * ✨ Run make clean after uninstall template (#244) * Run make clean after uninstall temp * Add flag * ✨Added message for terminal connection (#254) * Created flag for kernel templates * Add beta depot and flag to upgrade * Change --pros-4 to --beta * Add ReleaseChannel enum to conductor.py * Fix up some lines * Fixed making PROS 4 when trying to make PROS 3 project * Change beta enum to bool * WIP fix * Fix conductor.config issue * Remove debug prints * Allow uninstalling invalid templates * Change url * Fix url * Random line left lol * Fixing * Remove irrelevant change * Put this back * Add liblvgl message --------- Co-authored-by: Andrew Chen <chen4192@purdue.edu> Co-authored-by: Christopher Tan <72955765+Christopher-Tan@users.noreply.github.com> Co-authored-by: Kunwar Sahni <kunwar.sahni01@gmail.com> Co-authored-by: ayushuk <ayushshukla30339@gmail.com> Co-authored-by: Ayush Shukla <71904196+ayushuk@users.noreply.github.com>
noam987
approved these changes
Apr 18, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
set_variable
andread_variable
#259