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

Added --list option for apex run and updated readme/descriptions #21

Merged
merged 4 commits into from
Jan 23, 2023

Conversation

jsoverson
Copy link
Contributor

This PR

  • adds apex run --list and apex list tasks to list out tasks
  • centralized graphical table generation now that it was in a few places
  • updated the readme and some descriptions to call out the major parts of the apex CLI.
  • replaced justfile with apex.yaml and updated GH workflow to use apex test
  • added descriptions to task definitions to make prettier --list output
  • updated cliffy

apex run --list output:

Screen Shot 2023-01-20 at 8 17 47 PM

@jsoverson jsoverson requested a review from pkedy January 21, 2023 01:26
Copy link
Member

@pkedy pkedy left a comment

Choose a reason for hiding this comment

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

LGTM with 1 comment

README.md Outdated
To install from source, clone this repository and run `./apex run install`

```sh
git clone git@github.com:apexlang/apex.git && cd apex && ./apex install
Copy link
Member

@pkedy pkedy Jan 21, 2023

Choose a reason for hiding this comment

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

I recommend changing the link to https://github.com/apexlang/apex.git because I think that is more commonly used. Also, having each command on a separate line might be more Windows-friendly (except for the last command that runs a bash script. Might need WSL for that.)

git clone https://github.com/apexlang/apex.git
cd apex
./apex install

@jsoverson jsoverson merged commit ad7c563 into main Jan 23, 2023
@jsoverson jsoverson deleted the list-command branch January 23, 2023 15:14
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