Skip to content
This repository was archived by the owner on Sep 1, 2022. It is now read-only.

Conversation

@hectorsector
Copy link
Contributor

This PR fleshes out the explanations to a point where most folks should be able to follow them.

@github-learning-lab
Copy link

Your draft course version was created!


You can edit it using the Course Builder UI!

@github-learning-lab github-learning-lab bot had a problem deploying to production November 1, 2019 16:32 Failure
@github-learning-lab
Copy link

🔴 There was an error parsing your config.yml file.

YAMLException: can not read a block mapping entry; a multiline key may not be an implicit key at line 91, column 15:
          operator: test
                  ^

@github-learning-lab github-learning-lab bot had a problem deploying to production November 1, 2019 16:37 Failure
@github-learning-lab
Copy link

🔴 There was an error parsing your config.yml file.

YAMLException: unknown escape sequence at line 90, column 16:
          left: "/\s*if:\s*contains\s*\(\s*github\. ... 
                   ^

@github-learning-lab github-learning-lab bot had a problem deploying to production November 1, 2019 16:37 Failure
@github-learning-lab
Copy link

🔴 There was an error parsing your config.yml file.

YAMLException: incomplete explicit mapping pair; a key node is missed; or followed by a non-tabulated empty line at line 97, column 42:
     ...    expected: the conditional `if: contains(github.event.pull_req ... 
                                         ^

@github-learning-lab github-learning-lab bot had a problem deploying to production November 1, 2019 16:38 Failure
@github-learning-lab
Copy link

🔴 There was an error parsing your config.yml file.

YAMLException: bad indentation of a sequence entry at line 102, column 3:
      - type: respond
      ^

@hectorsector
Copy link
Contributor Author

This probably needs a run through by someone and can then be merged!

@brianamarie
Copy link
Contributor

I went through this and fixed one tiny sentence thing from my original commits, and also some logic that was looking for the wrong file name.

Thank you @hectorsector for making this better!

I'm going to merge this, and make the course public. It still needs to be made open source, but it should be ready to take for participants! cc @mattdavis0351 @crichID @carolynshin @a-a-ron

@brianamarie brianamarie merged commit 4e20044 into master Nov 3, 2019
@brianamarie brianamarie deleted the full-explains branch November 3, 2019 11:35
@brianamarie
Copy link
Contributor

Running into some problems, but I'll open an issue in learning-lab.

Screen Shot 2019-11-03 at 12 35 49

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants