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

Fix typo and update guidance to point to dfeR package #43

Merged
merged 3 commits into from
Apr 2, 2024

Conversation

cjrace
Copy link
Contributor

@cjrace cjrace commented Mar 9, 2024

Overview of changes

Update references from the Analysts' Guide to point to dfeR for connecting to SQL.

Why are these changes being made?

We have a function in dfeR that we will maintain there, as well as a vignette that walks through connecting to SQL.

Detailed description of changes

As it says, plus:

  • Removed link to old C+C materials that just shows a blank page
  • Removed random HTML files that popped up in my Git changes, assuming they shouldn't have been there to start with

Issue ticket number/s and link

None.

Checklist before requesting a review

  • I have checked the contributing guidelines
  • I have checked for and linked any relevant issues that this may resolve
  • I have checked that these changes build locally
  • I understand that if merged into main, these changes will be publicly available

Signed-off-by: Cam Race <52536248+cjrace@users.noreply.github.com>
@cjrace
Copy link
Contributor Author

cjrace commented Mar 11, 2024

@cjrace cjrace marked this pull request as ready for review March 27, 2024 16:41
@cjrace
Copy link
Contributor Author

cjrace commented Mar 27, 2024

@cjrace cjrace merged commit f45bbc4 into main Apr 2, 2024
@cjrace cjrace deleted the update-sql-connection-dfer branch April 2, 2024 15:18
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