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
Tdd Update #516
Tdd Update #516
Conversation
scripts/tdd.js
Outdated
// Grab snippetData | ||
const fileData = fs.readFileSync(`${SNIPPETS_PATH}/${fileName}.md`, 'utf8'); | ||
const fileData = fs.readFileSync(`${activeOrArchive}/${fileName}.md`, 'utf8'); | ||
// Grab snippet Code blocks | ||
const fileCode = fileData.slice(fileData.indexOf('```js'), fileData.lastIndexOf('```') + 3); |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Also @Chalarangelo I previously ignored the As other projects do ~ expose their Example: So I am going to run |
As you can see by the latest commit
Now the |
I don't know much so don't get 😠 at my stupid question. Will you like typing all of the tests or test only the examples ? Because I see something like test in |
@kingdavidmartins Yep, I wanted to ask you if there was a higher purpose to ignoring the @kriadmin All snippets will get test cases added over time. We will probably have a guideline about that (like 20-25 test cases at most, except snippets that explicitly require more) and we will carefully add them to existing snippets over time. I think the PR is ready to merge now and we are set to close the related issue (#78) and start adding tests over time, right? |
@Chalarangelo Hahaha I guess I was mistaken. I actually thought you said you didn't want users exposed
Yup Yup. Pretty much #78 should is good to go & PR ready to get merged Hey @kriadmin 👋 😄 we all still learning so don't ever feel whatever you ask is or will ever be considered a stupid question. You have contributed a lot to the project as is & are more than capable. Pretty much what @Chalarangelo said. The only thing I will add is that every snippet example that is added to the snippet should be added to the test. So starting out all snippet test should have 2 tests out of the 20-25 added from the examples then the community can really build things out from there |
@kingdavidmartins Ok, I'll go have lunch, retest your branch one last time, merge, close #78 and then maybe write a few tests myself and play around with the process a bit. I will also update |
Ah, it already does run on the |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for any follow-up tasks. |
Description
Update
tdd
script to work with both directories. Active Snippets & Archived SnippetsWhat does your PR belong to?
Types of changes
Lodash Backlog
method.md
] or [ADD:method.md
]method
to the lodash backlog.Checklist: