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

Updated with new data for 11th July #94

Merged
merged 1 commit into from
Jul 11, 2024
Merged

Updated with new data for 11th July #94

merged 1 commit into from
Jul 11, 2024

Conversation

t-surtees
Copy link
Collaborator

Pull request overview

Updating dashboard with data for latest fortnight, covering 17th-28th June.

Pull request checklist

Please check if your PR fulfils the following:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Tests have been run locally and are passing (run_tests_locally())
  • Code is styled according to tidyverse styling (checked locally with tidy_code())

What is the current behaviour?

Updated data for latest fortnight

What is the new behaviour?

Standard code changes and trends

Anything else

Add any notes for people reviewing and testing your code that are appropriate. Tag a @person to review if someone in particular needs to see this.

@t-surtees t-surtees requested a review from c-h-rowe July 11, 2024 08:04

# Install each package using install.packages()
# for (package in packages_to_install) {
# print(paste("Installing package:", package))

Check notice

Code scanning / lintr

Commented code should be removed. Note

Commented code should be removed.
# Install each package using install.packages()
# for (package in packages_to_install) {
# print(paste("Installing package:", package))
# install.packages(package)

Check notice

Code scanning / lintr

Commented code should be removed. Note

Commented code should be removed.
# }

# Install "dfeshiny"
# renv::install("dfe-analytical-services/dfeshiny")

Check notice

Code scanning / lintr

Commented code should be removed. Note

Commented code should be removed.
@rmbielby
Copy link
Contributor

Hey, just to let you know I'm not in the office yet today. Charlotte and Jen should be online and able to support if you need anything.

Copy link
Contributor

@c-h-rowe c-h-rowe left a comment

Choose a reason for hiding this comment

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

Approved for publication today.

@t-surtees
Copy link
Collaborator Author

Merge with main.

@t-surtees t-surtees merged commit 3c1f412 into main Jul 11, 2024
4 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.

3 participants