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

✨Add Icon and After support to upload_options #200

Merged
merged 1 commit into from
Feb 21, 2022

Conversation

BennyBot
Copy link
Member

Summary:

Makes --icon and --after work in upload_options in project.pros

Motivation:

Changes to vscode's project.pros views weren't working, this fixes it

References (optional):

purduesigbots/pros-vsc#42

Test Plan:

I tested it with the vs code extension and cli and it works.

Copy link
Member

@kunwarsahni01 kunwarsahni01 left a comment

Choose a reason for hiding this comment

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

LGTM

@kunwarsahni01 kunwarsahni01 merged commit 42853c2 into release/3.3.0 Feb 21, 2022
kunwarsahni01 added a commit that referenced this pull request Feb 21, 2022
* 🐛 emit correct compiler for C files when generating CDB (#188)

* Update Azure Pipelines with new self hosted M1 Mac (#189)

* Update azure-pipelines.yml for Azure Pipelines

* Update azure-pipelines.yml

* Update azure-pipelines.yml for Azure Pipelines

* Update azure-pipelines.yml for Azure Pipelines

* Fixes 'make.exe.exe' typo. Adds disable sentry prompt option for VSCode. (#190)

* Fixes 'make.exe.exe' typo. Adds disable sentry prompt option for VSCode.

* Fix sentry-off flag

* change sentry-off to no-sentry

* Update Version (#186) (#191)

* ✨Add Analytics (#193)

* 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

* ✨More Upload Options (#194)

* 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

* Update Version (#186) (#195)

* Update Version to 3.3.0

* 🐛Use Correct Linked Remote Name for Upload (#199)

* ✨Add Icon and After support to upload_options (#200)

* Change no internet exception to warning.

Co-authored-by: Alex Brooke <[email protected]>
Co-authored-by: BennyBot <[email protected]>
Co-authored-by: Benjamin Davis <[email protected]>
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