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

Fix issue where and keyword is wrongly used to determine quantity + new demo website #172

Merged
merged 7 commits into from
Jan 21, 2025

Conversation

jlucaspains
Copy link
Owner

@jlucaspains jlucaspains commented Jan 19, 2025

This pull request includes significant changes to the project structure and configuration, primarily focusing on replacing the demo directory with demo2 and updating related configurations accordingly. Additionally, it involves the removal of various configuration files and dependencies.

Fixes #171

Project Structure and Configuration Updates:

  • .github/workflows/demo.yml: Updated paths and commands to reflect the renaming of the demo directory to demo2, including switching from npm to yarn for dependency management and build processes. [1] [2]

Removal of old demo files

  • Deleted several configuration files related to ESLint, Prettier, and project setup, including .eslintignore, .eslintrc.cjs, .npmrc, .prettierignore, .prettierrc, README.md, package.json, app.html, +layout.svelte, +page.js, +page.svelte, styles.css, robots.txt, svelte.config.js, and vite.config.js in the demo directory. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15]

Updates to .gitignore and .npmrc:

  • demo2/.gitignore: Renamed from demo/.gitignore and updated to include additional output directories and environment files.
  • demo2/.npmrc: Added engine-strict=true to enforce strict engine compatibility.

Updates to .prettierignore:

@jlucaspains jlucaspains merged commit 240c8b8 into main Jan 21, 2025
3 checks passed
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.

getQuantity not working correctly for ingredients without a quantity
1 participant