Skip to content

Minor refresh#46

Merged
Keats merged 7 commits intoKeats:masterfrom
chevdor:master
Jul 3, 2021
Merged

Minor refresh#46
Keats merged 7 commits intoKeats:masterfrom
chevdor:master

Conversation

@chevdor
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
description = "A simple way to get started with a project"
authors = ["Vincent Prouillet <hello@vincentprouillet.com>"]
license = "MIT"
description = "A simple way to get started with a project by scaffolding from a template powered by the tera engine"
Copy link
Copy Markdown
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
Copy Markdown
Owner

Keats commented Jun 30, 2021

Any big features missing for your usage?

@chevdor
Copy link
Copy Markdown
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
Copy Markdown
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
MarlonBando pushed a commit to MarlonBando/my_nvim that referenced this pull request Sep 28, 2025
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