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

Writing and Visualising separation #5

Merged
merged 5 commits into from
Oct 6, 2023
Merged

Conversation

cjrace
Copy link
Contributor

@cjrace cjrace commented Oct 5, 2023

Overview of changes

Breaking out the writing and visualising page across the EES page, and 3 new pages:

  1. Writing about data
  2. Visualising data
  3. Publication scrums

I've also added a link to the RAP for stats guidance in the statistics production section so that points to the RAP for stats page from the RAP section given that people may look for it in either place.

One final thing is that I've also 'corrected' (hopefully!) the title to be Analysts' Guide - happy to be corrected on this if I've got it wrong!

It looks like a lot of changes but most of it was copy / paste, with some deletions of unnecessary / duplicating sections along the way. There's a few bits of original content, though these are mainly at the top of the 3 new pages and in certain sections to help them read easier to new readers.

Why are these changes being made?

As the existing page had a mesh of guidance, and was too long and unwieldy to be helpful.

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

@chfoster
Copy link
Contributor

chfoster commented Oct 6, 2023

These names don't line up rn

image

@chfoster
Copy link
Contributor

chfoster commented Oct 6, 2023

While I'm checking these names of the subsections don't line up either?
image

@cjrace cjrace merged commit f8d35aa into main Oct 6, 2023
@cjrace cjrace deleted the ees-visualing-separation branch October 6, 2023 08:28
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