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

Minor refresh #46

Merged
merged 7 commits into from
Jul 3, 2021
Merged

Minor refresh #46

merged 7 commits into from
Jul 3, 2021

Conversation

chevdor
Copy link
Contributor

@chevdor chevdor commented Jun 30, 2021

This PR fixes a few linter grumbles and mainly adds keywords for a better discovery.
I wish I found this project/repo earlier :)

Copy link
Owner

@Keats Keats left a comment

Choose a reason for hiding this comment

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

Looks good overall but I don't get why you added the GH actions file, Azure pipelines is already pretty fast.

- name: Cargo check
uses: actions-rs/cargo@v1
with:
command: check
Copy link
Owner

Choose a reason for hiding this comment

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

why add that compared to the current pipeline?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did not notice your pipeline at first :)
I also cannot run your pipeline in my fork (beside when opening a PR) so the github workflow makes it easier to check on my end.

I can remove it from the PR if you prefer.

Copy link
Owner

Choose a reason for hiding this comment

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

yes please, no need for 2 CI setup

Cargo.toml Outdated
authors = ["Vincent Prouillet <[email protected]>"]
license = "MIT"
description = "A simple way to get started with a project by scaffolding from a template powered by the tera engine"
Copy link
Owner

Choose a reason for hiding this comment

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

Tera with a capital T

@Keats
Copy link
Owner

Keats commented Jun 30, 2021

Any big features missing for your usage?

@chevdor
Copy link
Contributor Author

chevdor commented Jun 30, 2021

Any big features missing for your usage?

Not at that point but I did not test extensively yet. I will happily report back or open a PR if so. Thanks for asking !

@chevdor
Copy link
Contributor Author

chevdor commented Jun 30, 2021

Any big features missing for your usage?

Since you asked :) #48

@Keats Keats merged commit ea73816 into Keats:master Jul 3, 2021
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