-
Notifications
You must be signed in to change notification settings - Fork 1
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
Development #30
Development #30
Conversation
…ak-rule Disable eslint linebreak rule
feat(eslintrc.cjs): add support for YAML files chore(commitlint.config.js): add commitlint configuration chore(package.json): add commit script chore(App.jsx): refactor App component to use Nav component chore(posts.js): add getPost function to fetch individual post details chore(main.jsx): remove unused imports feat(Home.jsx): create Home component feat(PostDetail.jsx): create PostDetail component feat(Posts.jsx): create Posts component feat(index.jsx): create Nav component to handle routing The eslint configuration has been updated to include support for the latest ECMAScript version and module syntax. The ignore patterns have been modified to exclude additional files and directories. The rules for YAML files have been added to the eslint configuration. A commitlint configuration file has been added to enforce conventional commit messages. A commit script has been added to the package.json file to
…nt-config Put quotes around linebreak-style value
…mits to be more informative This will improve the teams' commits to the repo
… ones added changelog using standard-version package
…or future feature
Add mirageJS integration and mock placeholder data
refactor(posts): update import paths for get-posts and use-posts refactor(posts/components/post-card): update import path for react-router-dom refactor(posts/components/post-list): update import path for post-card component refactor(recipes/components/recipe-card): remove unused import refactor(recipes/components/recipe-list): remove unused import refactor(ui/Header): update import path for navLinks constant refactor(pages/About): remove unused import refactor(pages/PostDetail): update import path for GetPostById refactor(pages/Posts): update import path for PostList feat(pages/Recipes): add new RecipesPage component refactor(pages/index): update import path for Recipes refactor(routes/index): update import paths for pages and features/ui refactor(vite.config): add aliases for src directory and its subdirectories The changes were made to update import paths and add
…l.css' and moved to folder
…include a commit method Also include is a method to bump up version releases
…ak-style--rule Update linebreak-style to default to unix
getting a failure to load config from `/opt/build/repo/vite.config.js` and saying 'build.command' failed BREAKING CHANGE: the netlify git hooks fail
Feat/impromate browser router got netlify git hook to pass by deleting node_modules and package-lock.json. then reinstalling with `npm install`
✅ Deploy Preview for yumyumyes ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for yumyumyesdev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This runs fine for me. No eslint warnings (red squiggles) when I open a file on a Windows dev machine.
However, there is a very minor error in the browser console stating that the vite.svg
is not found because it has moved to the src/assets
folder.
merge latest version of development branch to main