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

✨ Feature ➾ Add .gitignore file upon taq init #1585

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

jchenche
Copy link
Contributor

@jchenche jchenche commented Dec 7, 2022

🌮 Taqueria PR

#Fixes #1461

🪁 Description

Add .gitignore file upon taq init.

🪂 Pre-Merge Checklist (Definition of Done)

🚦 Required to merge:

  • ⛱️ I have completed this PR template in full and updated the title appropriately
  • ⛵ My code builds cleanly, and I have manually tested the changes
  • 🏄‍♂️ Another team member has built this branch and done manual testing on the change
  • 🏖️ New and existing unit tests pass locally and in CI
  • 🔱 The test plan has been implemented and verified by an SDET
  • 🦀 Automated tests have been written and added to this PR
  • 🐬 I have commented my code, particularly in hard-to-understand areas
  • 🤿 Corresponding changes have been made to all documentation
  • 🐚 Required changes to the VScE have been made
  • 🪸 Required updates to scaffolds have been made
  • 🚢 The release checklist has been completed

🛩️ Summary of Changes

Please include a summary of the changes to the codebase. Please also include relevant motivation and context for the change
(e.g. what was the existing behaviour, why did it break, etc.)

🎢 Test Plan

Please describe the testing strategy and plan for this PR. Keep this lightweight and anticipate any testing challenges

🛸 Type of Change

  • 🧽 Chore ➾
  • 🛠️ Fix ➾
  • ✨ Feature ➾
  • 👷 Refactor ➾
  • 🧪 Pre-Release ➾
  • 🚀 Release ➾

@jchenche jchenche requested a review from a team December 7, 2022 08:18
@jchenche jchenche self-assigned this Dec 7, 2022
@jchenche jchenche changed the title Add ignore upon taq init ✨ Feature ➾ Add .gitignore file upon taq init Dec 7, 2022
@egarson
Copy link
Contributor

egarson commented Dec 7, 2022

LGTM

@egarson egarson merged commit 4e94adf into prerelease-0.25.0 Dec 7, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 7, 2022
@@ -425,6 +430,7 @@ const initProject = (
mkInitialDirectories(projectDir, maxConcurrency, i18n),
chain(_ => exec('npm init -y 2>&1 > /dev/null', {}, false, projectDir)),
chain(_ => exec('taq install @taqueria/plugin-core 2>&1 > /dev/null', {}, false, projectDir)),
map(_ => createGitIgnoreFile(projectDir)),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this be chain?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No because createGitIgnoreFile is a function that returns a future

@jchenche jchenche deleted the 1461_gitignore_on_init_JCC branch December 8, 2022 00:18
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