Skip to content

Conversation

capnrefsmmat
Copy link
Contributor

We neglected to say this explicitly, so the only way to find out is to check the codebook spreadsheet. I often find survey items by searching the coding page, so it's useful to be explicit here.

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

We neglected to say this explicitly, so the only way to find out is to check the codebook spreadsheet. I often find survey items by searching the coding page, so it's useful to be explicit here.
@capnrefsmmat capnrefsmmat requested a review from nmdefries March 28, 2022 18:55
@krivard
Copy link
Contributor

krivard commented Mar 28, 2022

(failing build is unrelated to this change)

Copy link
Contributor

@nmdefries nmdefries left a comment

Choose a reason for hiding this comment

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

👍

@capnrefsmmat capnrefsmmat merged commit 2e12abb into dev Mar 29, 2022
@capnrefsmmat capnrefsmmat deleted the survey/v15c branch March 29, 2022 14:23
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