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 build subcommand for building project #1

Merged
merged 5 commits into from
Sep 26, 2018

Conversation

snideto
Copy link
Collaborator

@snideto snideto commented Sep 21, 2018

Move build functionality from grid-tied.

@altendky altendky changed the title Feature/build Add build subcommand for building project Sep 21, 2018
<natures>
<nature>org.python.pydev.pythonNature</nature>
</natures>
</projectDescription>
Copy link
Owner

Choose a reason for hiding this comment

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

I'll let adding these slide. :]

Copy link
Owner

@altendky altendky left a comment

Choose a reason for hiding this comment

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

build_type_choices probably belongs in api and should become an enumeration now that it can be selected by something other than CLI text.

@snideto
Copy link
Collaborator Author

snideto commented Sep 25, 2018

pallets/click#605

@snideto
Copy link
Collaborator Author

snideto commented Sep 25, 2018

github.com/epcpower/ccstudiodss/commit/0641a7d5924b5ba1686f6b7f70ec881b1b6e998d

@altendky
Copy link
Owner

altendky commented Sep 25, 2018

Sorry, I see the click link now. Here's the basic conversion but I'll gander through the link for something niftier. Maybe an actual click type.

build_type_strings = [build_type.name for build_type in Build types]

build_type = BuildTypes[chosen_string]

@altendky
Copy link
Owner

Yeah, no named tuple. Stdlib Enum or enum34 from PyPI. Either PR for Click similar to the last suggestion in the ticket or something like I have above. I haven't contributed to click so I don't know how the process goes with them.

@altendky
Copy link
Owner

Note that GitHub inserts the commits into the history here so no need to link them.

@altendky altendky merged commit f319d40 into altendky:develop Sep 26, 2018
@epcAnker epcAnker deleted the feature/build branch April 27, 2022 17:12
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