Bump stefanzweifel/git-auto-commit-action from 4 to 5 #846
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![dependabot[bot]](https://avatars.githubusercontent.com/in/29110?s=60&v=4){kind=small}
Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 4 to 5. - [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases) - [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md) - [Commits](stefanzweifel/git-auto-commit-action@v4...v5) --- updated-dependencies: - dependency-name: stefanzweifel/git-auto-commit-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
dependabot
bot
requested review from
a team,
Jennit07 and
shintoLamp
and removed request for
a team
@check-spelling-bot Report🔴 Please reviewSee the 📂 files view, the 📜action log, or 📝 job summary for details. Unrecognized words (13)To accept these unrecognized words as correct, you could run the following commands... in a clone of the [email protected]:Public-Health-Scotland/source-linkage-files.git repository curl -s -S -L 'https://raw.githubusercontent.com/check-spelling/check-spelling/main/apply.pl' |
perl - 'https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/6445935597/attempts/1'OR To have the bot accept them for you, reply quoting the following line: Available 📚 dictionaries could cover words (expected and unrecognized) not in the 📘 dictionaryThis includes both expected items (232) from .github/actions/spelling/expect.txt and unrecognized words (13)
Consider adding them (in with:
extra_dictionaries:
cspell:aws/aws.txt
cspell:filetypes/filetypes.txt
cspell:software-terms/dict/softwareTerms.txt
cspell:k8s/dict/k8s.txt
cspell:npm/dict/npm.txtTo stop checking additional dictionaries, add (in check_extra_dictionaries: ''Errors (3)See the 📂 files view, the 📜action log, or 📝 job summary for details.
See ❌ Event descriptions for more information. If the flagged items are 🤯 false positivesIf items relate to a ...
|
Jennit07
approved these changes
Oct 20, 2023
Jennit07
deleted the
dependabot/github_actions/stefanzweifel/git-auto-commit-action-5
branch
Jennit07
added a commit
that referenced
this pull request
Nov 21, 2023
* Rename function to `convert_sc_sending_location_to_lca` (#839) * Bump `{slfhelper}` version The new version is needed to read the SLFs now. We use this in `get_existing_data_for_tests()` * Remove unnecessary code from `get_anon_chi` (#759) * remove unnecessary code from `get_anon_chi` `get_anon_chi` was updated in slfhelper v0.10 * [check-spelling] Update metadata Update for https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/5669528966/attempts/1 Accepted in #759 (comment) Signed-off-by: check-spelling-bot <[email protected]> --------- Signed-off-by: check-spelling-bot <[email protected]> Co-authored-by: marjom02 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * Set the default reporter for `tar_outdated()` and friends * Comment out dataset writing targets These take a very long time to run, so were skipped at the last update. They need to be revisited. * Make sure `year` is added as the first variable * Correct some documentation (#769) * Correct some documentation This resolves a build warning. * Style code --------- Co-authored-by: Moohan <[email protected]> * Make some changes suggested by lintr Lots of layout changes, as well as lots of implicit to explicit integer / double changes. * Document * Fix documentation typo * Investigate missing datazone from episode file (#773) * Format postcode into `pc7` format * Style code * Style code * Update documentation * Update comment in R/process_extract_ae.R * Implement catch-all for PC7 format --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: James McMahon <[email protected]> Co-authored-by: Moohan <[email protected]> * Remove some obsolete code (#770) * Remove some obsolete code Renaming and removing some functions. * Style code --------- Co-authored-by: Moohan <[email protected]> Co-authored-by: Zihao Li <[email protected]> * Simplify `create_hscp_test_flags` (#772) * Simplify `create_hscp_test_flags` * Update documentation * Style code * simplify `create_hb_test_flags` * implement hscp test flags into tests * Simplify `create_demog_test_flags` --------- Co-authored-by: James McMahon <[email protected]> Co-authored-by: Moohan <[email protected]> * Rewrite case when statements (#780) * updated code from case_when to case_match as it's a bit easier to read * Style code * changed some more `case_when` to `case_match` * Style code * [check-spelling] Update metadata Update for https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/5752014211/attempts/1 Accepted in #780 (comment) Signed-off-by: check-spelling-bot <[email protected]> * Add tests for `convert_sending_location_to_lca` --------- Signed-off-by: check-spelling-bot <[email protected]> Co-authored-by: marjom02 <[email protected]> Co-authored-by: SwiftySalmon <[email protected]> Co-authored-by: James McMahon <[email protected]> * Update R-CMD-check.yaml (#781) Co-authored-by: Jennit07 <[email protected]> * added solve for hscp names (#789) In processed extract variable is called hscp, and in final SLF it's called hscp2018. Fixed with nested if statement Co-authored-by: marjom02 <[email protected]> * Fix locality (#802) Tiny error and a simple fix. Co-authored-by: Jennit07 <[email protected]> * Add simple scripts for running targets as a workbench job (#767) * Fix CHI duplicates of chi in individual file (#791) * fix duplicated matches in chi in sc data. * Update R/create_individual_file.R * update on join_sc_client * Create a test checking if individual files have duplicated chi * add duplicated chi number to the tests in process_tests_individual_file --------- Co-authored-by: lizihao-anu <[email protected]> Co-authored-by: James McMahon <[email protected]> * Update NSU code for new 22/23 cohort (#784) Update `check_year_valid` for NSUs * Amend `get_boxi_extract_path` function for archiving DN and CMH data (#785) * Update `get_boxi_extract_path` for DN/CMH data * Remove extra function * [check-spelling] Update metadata Update for https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/5856792420/attempts/1 Accepted in #785 (comment) Signed-off-by: check-spelling-bot <[email protected]> --------- Signed-off-by: check-spelling-bot <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: James McMahon <[email protected]> * Fix increase in total preventable beddays (#779) * further obsolete code change * fix the preventable_beddays Co-authored-by: James McMahon <[email protected]> --------- Co-authored-by: James McMahon <[email protected]> Co-authored-by: Jennit07 <[email protected]> * fix warning on `:=` (#797) * fix warning on `:=` * Update R/aggregate_by_chi.R Co-authored-by: James McMahon <[email protected]> * Style code --------- Co-authored-by: James McMahon <[email protected]> Co-authored-by: lizihao-anu <[email protected]> * Add 2324 targets/workbench job file * Use `get_source_extract_path` in homelessness (#796) This was already set up, just not used for some reason. Note that this will switch from using a `.rds` to `.parquet` (unless you do `get_source_extract_path(year, "Homelessness", ext = "rds")`). Co-authored-by: Jennit07 <[email protected]> * Correct tests for NSU * Update script for extracting NSU from SMRA space * Update year in 99_NSU extract script * Update news for September 23 update (#811) * Update News for March and June updates * Update release date * WIP - update news for Sep update * Update NEWS.md Fix some typos / grammar --------- Co-authored-by: James McMahon <[email protected]> * Apply styling * Fix issue with `case_match` types (#810) * Fix issue with `case_match` types It seems that `case_match()` is stricter about types than `case_when()`. See the below code: ```r library(dplyr) # Breaks mutate(starwars, new_height = case_when( height == "172" ~ "170"), new_height2 = case_match( height, "172" ~ "170" ), .after = "height" ) # Works mutate(starwars, new_height = case_when( height == "172" ~ "170"), new_height2 = case_match( height, 172L ~ "170" ), .after = "height" ) ``` Since `sending_location` is an integer, the LHS of `case_match` must be numeric. It was slightly incorrect previously but `case_when` let us get away with it! I also updated and added to the tests. * Style code * Style code --------- Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Bug - Outpatients tests failing due to missing HSCP (#816) * Update `produce_source_extract_tests` * Update outpatients tests with hscp_var = FALSE * Revert "Style code" This reverts commit 8e73d4a. * Style code * simplify code * Update documentation * Rename `hscp_var` to `add_hscp_count` * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: James McMahon <[email protected]> Co-authored-by: Moohan <[email protected]> * fix read_sc_all_alarms_telecare with incorrect format in period (#814) * fix read_sc_all_alarms_telecare with the incorrect format in period --------- Co-authored-by: lizihao-anu <[email protected]> Co-authored-by: James McMahon <[email protected]> * Fix `convert_sending_location_to_lca` example * Use `col_select` instead of `columns` in tests * Add tests for `compute_mid_year_age` (#809) * Add tests for `compute_mid_year_age` * Remove redundant code * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Add a new function to set up keyring (#800) * Add a new function to set up keyring I've tested this by deleting my `.Renviron` and deleting my keyring `keyring::keyring_delete("createslf")` and it seems to work. Would be great to have someone with an existing set-up (Jen) test it, and to have someone who doesn't have it set up to test it. The code looks complicated but I've just tried to catch every scenario, so the process should be smooth and clear (from the user's point of view). I've also expanded the code relating to the username, which will now hopefully work in more cases. * [check-spelling] Update metadata Update for https://github.com/Public-Health-Scotland/source-linkage-files/actions/runs/5824423711/attempts/1 Accepted in #800 (comment) Signed-off-by: check-spelling-bot <[email protected]> * Update documentation --------- Signed-off-by: check-spelling-bot <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Add additional tests for `get_file_path` (#808) * Add additional tests for `get_file_path` * Style code --------- Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Rename `run_episode_file()` -> `create_episode_file()` (#803) * Rename `run_episode_file()` -> `create_episode_file()` This improves consistency! When speaking to Megan we noted that having the two 'main' functions with different names was needlessly confusing! * Delete run_targets_tests.R * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * Remove incorrect references to rds (#798) * Remove incorrect references to rds Since we (mostly) don't use rds anymore these references are incorrect and potentially confusing. I've updated lots of documentation to remove the reference to rds. I've also updated many comments that mentioned rds (these were probably the most confusing). * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * Make targets and tarchetypes required packages (#799) Co-authored-by: Megan McNicol <[email protected]> * Update episode file functions to pass data through (#754) * Update `read_file` to return an empty tibble if passed the dummy path This is needed for some other bits, notably NSUs * Update SPARRA and HHG paths to return dummy if the year is invalid * Extract all data as a parameter * Style code * Update documentation * Style code * Update documentation * rename `run` to `create_episode_file` * Update documentation --------- Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennifer Thom <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Tests/it extract path (#807) * Add additional tests for `check_it_reference()` * Make the check on the IT reference stricter * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Update workflow to run against the development branch (#795) * Make test-coverage.yaml run against development * Make lint-changed-files.yaml run against development --------- Co-authored-by: Jennit07 <[email protected]> * Remove package wide imports of `readr` (#792) * Update documentation * Use `readr::` where possible * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * Handle OpenData extracts better (#794) * Refactor the LA Code OpenData This should now run as its own target and then be passed to the homelessness data. I also added some tests. * Also add some tests for the GP prac clusters OpenData * Update documentation --------- Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Fix the pkgdown site/job (#804) * Fix the pkgdown site/job It generates this site: https://public-health-scotland.github.io/source-linkage-files/ although it hasn't been working for a while since any new function needs to be added to (or captured by) the `_pkgdown.yml` file. This PR is a pretty minimal fix to get the site working again. * Update documentation * Update documentation * Update `create_episode_file` * Remove `run_episode_file` * update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Jennifer Thom <[email protected]> * Add new 'final' file path functions (#787) * New function for SLF final file paths * Implement final file path functions * Style code * Update documentation * Update final file paths to use `...` * fixing conflicts with `run episode file` getting renamed to `create episode file` * Update documentation * Update documentation * Style code --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: marjom02 <[email protected]> Co-authored-by: SwiftySalmon <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * Check scripts are in snake case (#793) * Update `get_boxi_extract_path` for DN/CMH data * Remove extra function * Update documentation * change `get_boxi_extract_path` to snake_case * change `get_source_extract_path` to snake_case * Update documentation * Update targets with snake_case * Fix typo * Style code --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: James McMahon <[email protected]> Co-authored-by: Megan McNicol <[email protected]> Co-authored-by: SwiftySalmon <[email protected]> * transform the python script for sorting BI extracts to R (#833) * transform the python script for sorting BI extracts to R * Style code * Delete 00-Sort_BI_Extracts.py --------- Co-authored-by: lizihao-anu <[email protected]> * Use `get_slf_episode_path` in right place * fix pipe * Fix typo in string * Update documentation * Rename to `convert_sc_sending_location_to_lca` * Update documentation * Style code * Update documentation --------- Signed-off-by: check-spelling-bot <[email protected]> Co-authored-by: James McMahon <[email protected]> Co-authored-by: Megan McNicol <[email protected]> Co-authored-by: marjom02 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Zihao Li <[email protected]> Co-authored-by: lizihao-anu <[email protected]> * Rename function `add_smrtype` (#840) * rename to `add_smrtype` * Rename script to `add_smrtype` * update documentation * Remove TODO comment * Style code * Update documentation --------- Co-authored-by: Jennit07 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> * add_keep_population_flag for create_individual_file * fix homelessness path * Update documentation * Style code * change boxi file names back to capitals (#845) A previous pull request changed all capitals to lowercase - however boxi file names have capitals so it was no longer reading in files. This is a fix Co-authored-by: marjom02 <[email protected]> Co-authored-by: Jennit07 <[email protected]> * Bump stefanzweifel/git-auto-commit-action from 4 to 5 (#846) Bumps [stefanzweifel/git-auto-commit-action](https://github.com/stefanzweifel/git-auto-commit-action) from 4 to 5. - [Release notes](https://github.com/stefanzweifel/git-auto-commit-action/releases) - [Changelog](https://github.com/stefanzweifel/git-auto-commit-action/blob/master/CHANGELOG.md) - [Commits](stefanzweifel/git-auto-commit-action@v4...v5) --- updated-dependencies: - dependency-name: stefanzweifel/git-auto-commit-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Apply suggestions from code review Co-authored-by: Jennit07 <[email protected]> * Style code * some changes to add_keep_population_flag * Style code * Update documentation * some changes to add_keep_population_flag * Style code * minor changes * Update documentation --------- Signed-off-by: check-spelling-bot <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: James McMahon <[email protected]> Co-authored-by: Megan McNicol <[email protected]> Co-authored-by: marjom02 <[email protected]> Co-authored-by: Megan McNicol <[email protected]> Co-authored-by: Moohan <[email protected]> Co-authored-by: Jennit07 <[email protected]> Co-authored-by: lizihao-anu <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Jennifer Thom <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumps stefanzweifel/git-auto-commit-action from 4 to 5.
Release notes
Sourced from stefanzweifel/git-auto-commit-action's releases.
... (truncated)
Changelog
Sourced from stefanzweifel/git-auto-commit-action's changelog.
Commits
8756aa0Update node version to node20 (#300)17a44b0Merge branch 'master' of github.com:stefanzweifel/git-auto-commit-action43818d5Fix Typo1094465Use actions/checkout v4 in examples3d1b5e0Bump actions/checkout from 3 to 4 (#302)47a8ad5Bump bats from 1.9.0 to 1.10.0 (#293)77a7b3fBump github/super-linter from 4 to 5 (#289)9cc0a1fSeems like there is an extra space (#288)0b5f8a5Update Test3a446b7Merge branch 'master' of github.com:stefanzweifel/git-auto-commit-actionDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)