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
Are updates to "Install and Require Helmet" challenge needed? #39582
Comments
If changing the tests to allow |
Just bringing this back up: Are we changing the tests, or changing the instructions to be explicit about the versioning? |
Personally, I'd say both. Show the npm command for how to install a specific package version and make the test ignore/accept the caret. |
@lasjorg I like that. This kind of lesson is so short, I also think it would be useful to add the information about I am going to open this up for general help |
Hi, I do exactly like npm install --save-exact helmet@3.21.3 and helmet = require('helmet'); but still not pass the challenge, anyone, please help me, thanks |
Hey @flemmingbond21 , Without seeing your code, or being able to test your project, it will be difficult to assist. Please make use of the forum to open a topic, and provide as much detail as possible. It will be easier to assist that way. |
my code: // running tests |
@flemmingbond21 As @ShaunSHamilton mentioned, please use our forum for questions that involve debugging challenge or project solutions. Thank you. |
…elmet instead of let
Challenge:
https://www.freecodecamp.org/learn/information-security/information-security-with-helmetjs/install-and-require-helmet
https://github.com/freeCodeCamp/freeCodeCamp/blob/master/curriculum/challenges/english/09-information-security/information-security-with-helmetjs/install-and-require-helmet.english.md
Does this challenge need instructions on how to actually install the specified version? On repl.it if you search for a package it will just give you the latest version.
If you run
npm i helmet@3.21.3
it will add the package with the caret^
in front of the version number but the test does not allow for that. Should the test allow for the caret?The text was updated successfully, but these errors were encountered: