This repository has been archived by the owner on Feb 9, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
London 9 - Vitalina Kuzmenko - cyf-hotel-react #540
base: master
Are you sure you want to change the base?
London 9 - Vitalina Kuzmenko - cyf-hotel-react #540
Changes from all commits
e966460
2982a72
3f942e9
b35b5b4
2a9776b
667fb27
0ff6fe5
c7a07de
41c2b68
32553f1
9894190
c7c64d7
bd9bdfa
6f6253b
fdac8c7
8071614
55e52e8
94153da
e93ffd9
4623b34
c9c2ef0
ff7f13b
16b991f
7486e0d
b8c2b49
3b6f5aa
5cb8195
6d84c18
d507978
11c0f7f
82ae0ee
abc9da9
9134110
b94706f
50c04af
bac00d8
1337561
c68a883
c295161
467f3ff
9d25334
82d3fad
aa61fe9
aa43c8b
823ab65
4f6d2f7
526f186
aa96a2a
25ba080
c3e79fe
b2691b4
bdb2e0a
bbc691c
52e0928
7ad477a
b96ceaa
b443ff8
b586add
0e675ab
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
Do we we have our dependencies int he correct sections? 😃
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.
I don't know. I was installing dependencies using npm install. And it was installed automatically. Is something wrong with it?
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.
So there is just one issue, eslint should be included in the dev dependencies section.
There are two types of dependencies:
Eslint falls into the second one
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.
You can read more about this here: https://docs.npmjs.com/specifying-dependencies-and-devdependencies-in-a-package-json-file
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.
But I have it inside devDependencies, no?
"devDependencies": {
"eslint-plugin-react-hooks": "^4.6.0",
Maybe I will ask you personally about it :) Thank you!