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

Package app cleanup #18

Merged
merged 2 commits into from
Jan 15, 2025
Merged

Package app cleanup #18

merged 2 commits into from
Jan 15, 2025

Conversation

seankross
Copy link
Contributor

@seankross seankross commented Dec 6, 2024

Description

  • Adds basic GitHub Actions based CI/CD infrastructure and related badging.
  • Adds example_callback() to help folks who are deploying this package understand what data they need to provide to the app.
  • Adds oncarto_file() to help access files that are packaged with the app.
  • The county-incidence tab now uses the best-practice pattern of putting the conditional logic within the renderLeaflet function call.
  • Adds WILDS project status badging.
  • Modifies tests to use example_callback().

Example

oncarto::oncarto_file() |> basename() |> paste0(collapse = "\n") |> cat()
#> fh-contact-info.md
#> fh-logo.png
#> fh.css

Testing

All tests pass for me locally. The purpose of this PR is to set up automatic testing on GitHub.

@seankross seankross changed the base branch from main to package-app December 6, 2024 18:43
@seankross seankross marked this pull request as ready for review January 13, 2025 15:58
@seankross seankross requested a review from vsriram24 January 13, 2025 15:58
@vsriram24 vsriram24 merged commit 14a08e1 into package-app Jan 15, 2025
6 checks passed
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