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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: create fledge.yaml #520

Merged
merged 2 commits into from
Sep 6, 2022
Merged

ci: create fledge.yaml #520

merged 2 commits into from
Sep 6, 2022

Conversation

maelle
Copy link
Member

@maelle maelle commented Sep 1, 2022

@krlmlr btw fledge is super behind on version bumping 馃槄 (which is actually very handy for interactive testing but not the best example 馃槄 )

Comment on lines 22 to 26
- name: Install fledge
run: Rscript -e 'install.packages("fledge")'

- name: Bump version
run: Rscript -e 'fledge::bump_version(which = "dev")'
Copy link
Contributor

Choose a reason for hiding this comment

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

Will this work? Should we:

  • install from "."
  • add a flag to bump_version() ?

Copy link
Member Author

Choose a reason for hiding this comment

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

install from "."

oh right fledge repo itself is a special case.

add a flag to bump_version() ?

what do you mean?

Copy link
Contributor

Choose a reason for hiding this comment

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

fledge::bump_version(no_change_behavior = "noop")

@krlmlr krlmlr marked this pull request as ready for review September 6, 2022 09:39
@krlmlr krlmlr merged commit fa91628 into main Sep 6, 2022
@krlmlr krlmlr deleted the gha-daily branch September 6, 2022 09:39
@krlmlr
Copy link
Contributor

krlmlr commented Sep 6, 2022

Thanks! Let's see how this flies.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants