Skip to content
This repository was archived by the owner on Dec 23, 2021. It is now read-only.

Command Pallet Restructure (New File, Open Simulator), README change to be more concise - #284

Merged
Vandy Liu (vandyliu) merged 11 commits into
devfrom
users/t-vali/command-p-restructure
Mar 31, 2020
Merged

Vandy Liu (vandyliu) merged 11 commits into
devfrom
users/t-vali/command-p-restructure

Conversation

@vandyliu

@vandyliu Vandy Liu (vandyliu) commented Mar 30, 2020 •

Copy link
Copy Markdown
Contributor

AB#34298

Description:

  • Changed command pallet
  • Made README more concise
  • Added template file for CLUE (which will likely change in the future)
  • Will need to have a part 2 PR to make picture src actual links instead of relative file paths

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Limitations:

Please describe limitations of this PR

Testing:

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A
  • Test B

Checklist:

  • My code follows the style guidelines of this project
  • My code has been formatted with npm run format and passes the checks in npm run check
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

@vandyliu Vandy Liu (vandyliu) changed the title Command Pallet Restructure, README change to be more concise Command Pallet Restructure (New File, Open Simulator), README change to be more concise Mar 30, 2020

@isadorasophia Isa (isadorasophia) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey! That was really awesome! Thanks for the PR description 🌟

Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md
Comment thread README.md Outdated
Comment thread src/templates/clue_template.py
@vandyliu
Vandy Liu (vandyliu) merged commit 0e94c8a into dev Mar 31, 2020
@vandyliu
Vandy Liu (vandyliu) deleted the users/t-vali/command-p-restructure branch March 31, 2020 19:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants