cran-task-views / ctv

CRAN Task View Initiative
82 stars 13 forks source link

installing github action for automatic checks #40

Closed tuxette closed 2 years ago

zeileis commented 2 years ago

Nathalie @tuxette, thanks for putting this together. I did some touch-ups, mostly in formatting and documentation. (Disclaimer: I didn't try to actually run the action, though.) I just wondered whether it is true that the task-view input still has the required: true specification. I thought we were able to avoid that by computing it from the repository name? But possibly I'm not understanding correctly how exactly this is processed.

Feel free to merge the PR yourself if everything is in order (and I didn't mess up anything).

tuxette commented 2 years ago

@zeileis :

zeileis commented 2 years ago

OK, good, thanks Nathalie @tuxette. Additionally, I think it would be fair to add "CRAN Task View Initiative" as the second author because we did a few improvements.

tuxette commented 2 years ago

I removed the section on the task-view input and retested it. I think it's OK now and I'm merging this branch.

zeileis commented 2 years ago

Wonderful thanks! 🎉

tuxette commented 2 years ago

OK, good, thanks Nathalie @tuxette. Additionally, I think it would be fair to add "CRAN Task View Initiative" as the second author because we did a few improvements.

I forgot this remark. It's done now.