Curriculum: Revise `Check for Understanding` questions of for Data Visualization, list other fixes needed

Created on 27 Dec 2019  路  8Comments  路  Source: Techtonica/curriculum

  1. Revise the Check for Understanding section of our Data Visualization Outline based on this comment: https://github.com/Techtonica/curriculum/pull/1107#issuecomment-568230598
    For more details on what we're looking for in a Check for Understanding section, see the directions in #700.

  2. Implement any small things mentioned in the following comment: https://github.com/Techtonica/curriculum/issues/1086#issuecomment-567664466
    If there are fixes mentioned that seem a bit larger, leave a comment here on #1110 and we will make some more issues.

EASY elective gssoc20

Most helpful comment

@pa7ri This is because you should be making changes in a forked repo - you don't have permission to directly edit Techtonica/curriculum. It's ok, you can fix it in 5-10 min:

  1. enter git log in your terminal, and write down the commit(s) hashes from this issue only from oldest to newest. you only have to write down the first 7 digits. We are going to transfer these changes to a new repo soon. So for example, if you had this:
    image

    If you only want the latest two, you would write in a note to yourself:

    • 00eb81b
    • d7dfa51
  2. Fork the repo
    image

  3. git clone that forked repo

  4. cd into that forked repo.

  5. enter git cherry-pick <oldest hash you wrote down>, ex: git cherry-pick 00eb81b

  6. enter git cherry-pick <next-oldest hash you wrote down>, etc, until you've done them all.
  7. push up and make a pull request. the changes you want should be there. Let us know if this doesnt work.

All 8 comments

This is elective so bumping out of the TODO project

Is this issue be worked under GSSoC20 ? @alodahl @vegetabill

I'm a gssoc participant and interested in working on this issue, if possible :smiley:

@pa7ri No you could start working on this issue.

@pa7ri You only have 9 days left, if we don't get any response from your side, the issue will be assigned to someone else.

I think I've just completed what is requested but while trying to push my changes into a branch but it says : ERROR: Permission to Techtonica/curriculum.git denied to pa7ri. any help with this?

@pa7ri This is because you should be making changes in a forked repo - you don't have permission to directly edit Techtonica/curriculum. It's ok, you can fix it in 5-10 min:

  1. enter git log in your terminal, and write down the commit(s) hashes from this issue only from oldest to newest. you only have to write down the first 7 digits. We are going to transfer these changes to a new repo soon. So for example, if you had this:
    image

    If you only want the latest two, you would write in a note to yourself:

    • 00eb81b
    • d7dfa51
  2. Fork the repo
    image

  3. git clone that forked repo

  4. cd into that forked repo.

  5. enter git cherry-pick <oldest hash you wrote down>, ex: git cherry-pick 00eb81b

  6. enter git cherry-pick <next-oldest hash you wrote down>, etc, until you've done them all.
  7. push up and make a pull request. the changes you want should be there. Let us know if this doesnt work.

It worked! _(I was trying to push directly to this repo instead of fork it)_ Thank you very much @alodahl and @Kundan28

Was this page helpful?
0 / 5 - 0 ratings