jiebi/CodeConvo huggingface dataset:
Change:
Problem: Simple content description error. It says in the challenge description that when the user has a calorie deficit, the remainingCalories will be a negative (less than 0). It is actually the opposite. For example, if I were to input this in the form: The description of the step must say that it will be a negative value if the input is a surplus (above budget). ! Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the Happy coding

Discussion:
# --description-- <del> When the user has a calorie deficit, the `remainingCalories` value will be negative. You don't want to display a negative number in the result string. </del> <ins> When the user has a calorie surplus, the `remainingCalories` value will be negative. You don't want to display a negative number in the result string. </ins> `Math.abs()` is a built-in JavaScript method that will return the absolute value of a number.
Change:
It looks like a lot of people on the forums and discords are confused on the syntax for this step. we could add an updated description with code example to guide campers. js " " + "string" ` see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the Happy coding
Step 63 You should now see the same bunch of characters in your console. Your padRow function is doing the exact same thing you were doing earlier, but now it's in a reusable section of its own. Use the addition operator to concatenate a blank space " " to the beginning and end of your repeated character string. Remember that you can use the + operator to concatenate strings like this: " " + "string" This is what i got and it was accepted function padRow(rowNumber, rowCount) { return " " + (rowNumber) + " " }

Discussion:
Use the addition operator to concatenate a blank space `" "` to the beginning and end of your repeated `character` string. <ins> Remember that you can use the `+` operator to concatenate strings like this: ```js " " + "string" ``` </ins> # --hints-- You should concatenate an empty space to the beginning of your returned value.
Change:
Hey any more context on this issue? Seems pretty empty to be already opened with the label.
the challenge needs more tests, it's not a high priority thing, that comment has been there for a few years already

Discussion:
assert.deepEqual(tokenize(testStr2, '&', '@'), res2); ``` <ins> `tokenize('hello^|world|how^are^you^|', '|', '^')` should return `['hello|world', 'howareyou|']` ```js assert.deepEqual(tokenize(testStr3, '|', '^'), res3); ``` </ins> # --seed-- ## --after-user-code--

const testStr1 = 'one^|uno||three^^^^|four^^^|^cuatro|'; const res1 = ['one|uno', '', 'three^^', 'four^|cuatro', '']; <del> // TODO add more tests </del> const testStr2 = 'a@&bcd&ef&&@@hi'; const res2 = ['a&bcd', 'ef', '', '@hi']; <ins> const testStr3 = 'hello^|world|how^are^you^|'; const res3 = ['hello|world', 'howareyou|']; </ins> ``` ## --seed-contents--
Change:
! Frequently asked question segment is not aligned with the previous elements on the homepage to down to frequently asked questions the misalignment of frequently asked questions segment with the previous elements The frequently asked question segment should be aligned with the previous elements of the homepage for a better UI ! Device: Laptop OS: Windows 11 Browser: Brave Version: 1.64.122 No response
This issue is because the Certifications section has extra side padding: We can add a class to the FAQ section, and give it , too. I'm not sure if this change would collide with the changes you're making to the landing page. I'm also wondering if the Certification section being so wide (comparing to the one on /learn) is still desired. If it's not, I think we can take this opportunity to update the section.
Yes, we can standardize all of them. My changes are effecting the landing top section.
Thanks, Ahmad. I'm temporarily blocking this issue until the CSS situation is a little less messy. I'll get back to this and come up with a suggestion after is merged.
We will be making changes to the certification section in the near future, so I think we can keep the scope of this issue to just adding some extra padding to the FAQ section, as suggested here: This issue is open for contribution. We typically do not assign issues. Instead, we accept the most quality contribution followed by the one that is made first. Please make sure you read our as well as our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Happy contributing.

Discussion:
const faqItems = t<string, string & FaqItem[]>('landing.faqs'); return ( <del> <Col sm={8} smOffset={2} xs={10} xsOffset={1}> </del> <ins> <Col md={8} mdOffset={2} sm={10} smOffset={1} xs={12} className='faq-section' > </ins> <h2 className='big-heading'>{t('landing.faq')}</h2> <Spacer size='small' /> {faqItems.map((faq, i) => (

padding: 4vw 15px; } <del> .certification-section { </del> <ins> .certification-section, .faq-section { </ins> padding-inline: 4vw; }
Change:
first discussed on the forum here: I had a concern about the starting sentence of step 26 of the js pyramid project here: It starts out with: "The let keyword allows a variable to be reassigned." which I think can be misinterpreted to mean that we can use the keyword to reassign a variable. I believe a reword to "Declaring a variable with the let keyword allows it to be reassigned." will be not as easy to misinterpret. N/A slight reword to instructions needed as per above No response N/A No response
Seems like a good change. This looks like something that can be fixed by "first-time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: From: To: Please make sure you read our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.

Discussion:
# --description-- <del> The `let` keyword allows a variable to be reassigned. This means you could change `character` later to be a completely different value. </del> <ins> Declaring a variable with the `let` keyword allows it to be reassigned. This means you could change `character` later to be a completely different value. </ins> For this project, you will not want to change these variable values. So instead, you should use `const` to declare them. `const` variables are special.
Change:
Valid code will fail and give the wrong assert message about using which is confusing. Expected If you search for articles on you will find many of the examples has the min value inside the method, this includes MDN, fCC news articles, etc. The min value should be allowed inside the floor method. Update the regex that checks for to not depend on the rest of the code structure to avoid false positives. Tests: Valid code should pass and not throw unrelated assert messages. No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] Forum:

Discussion:
You should use `Math.floor()` to round the result down to the nearest whole number. ```js <del> assert.match(code, /consts+randomDices*=s*Math.floors*(s*Math.randoms*(s*)s**s*6s*)s*/); </del> <ins> assert.match(code, /consts+randomDices*=s*Math.floors*((?s*Math.randoms*(s*)s**s*6s*)s*/); </ins> ``` You should add `1` to the end result of your `randomDice` variable. ```js <del> assert.match(code, /consts+randomDices*=s*Math.floors*(s*Math.randoms*(s*)s**s*6s*)s*+s*1s*/); </del> <ins> assert.match(code, /consts+randomDices*=s*Math.floors*((?s*Math.randoms*(s*)s**s*6s*)s*+s*1)?s*/); </ins> ``` # --seed--
Change:
We are using the class to conditionally hide some text on mobile screens: This class comes from Bootstrap, which is something we are going to remove (). So instead of using the class, we can use an check, like this (which is also a little more consistent with how we've handled this kind of conditional rendering): Update the file to replace the class with an check. I noticed that we are using multiple methods for mobile device detection, so I think the check should be standardized, and there should be a shared util for this check. But I'm leaving this out of the scope of this issue, and will create a separate issue for it.
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
I already have a pull request that technically resolves this. While it is done the same way we did the lower-jaw i.e checks for a desktop instead of being mobile , it does solve the problem and pass the tests.

Discussion:
"update-email": "Update my Email", "verify-email": "Verify Email", "submit-and-go": "Submit and go to next challenge", <ins> "submit-and-go-2": "Submit and go to next challenge (Ctrl + Enter)", "submit-and-go-3": "Submit and go to next challenge (Command + Enter)", </ins> "go-to-next": "Go to next challenge", <ins> "go-to-next-2": "Go to next challenge (Ctrl + Enter)", "go-to-next-3": "Go to next challenge (Command + Enter)", </ins> "ask-later": "Ask me later", "start-coding": "Start coding!", "go-to-settings": "Go to settings to claim your certification",

import Progress from '../../../components/Progress'; import GreenPass from '../../../assets/icons/green-pass'; import { Spacer } from '../../../components/helpers'; <ins> import { MAX_MOBILE_WIDTH } from '../../../../config/misc'; </ins> import './completion-modal.css'; import callGA from '../../../analytics/call-ga';

const isMacOS = navigator.userAgent.includes('Mac OS'); <ins> const isDesktop = window.innerWidth > MAX_MOBILE_WIDTH; let buttonText; if (isDesktop) { if (isMacOS) { buttonText = isSignedIn ? t('buttons.submit-and-go-3') : t('buttons.go-to-next-3'); } else { buttonText = isSignedIn ? t('buttons.submit-and-go-2') : t('buttons.go-to-next-2'); } } else { buttonText = isSignedIn ? t('buttons.submit-and-go') : t('buttons.go-to-next'); } </ins> return ( <Modal onClose={close}

data-cy='submit-challenge' onClick={() => submitChallenge()} > <del> {isSignedIn ? t('buttons.submit-and-go') : t('buttons.go-to-next')} <span className='hidden-xs'> {isMacOS ? ' (Command + Enter)' : ' (Ctrl + Enter)'} </span> </del> <ins> {buttonText} </ins> </Button> <Spacer size='xxSmall' /> {this.state.downloadURL ? (
Change:
All links in the english curriculum need to be replaced. This is the last of the S3 links Here are the new CDN links that should be used instead multiple see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response

Discussion:
# --instructions-- <del> Time to take a break from Camper Cat and meet fellow camper Zersiax (@zersiax), a champion of accessibility and a screen reader user. To hear a clip of his screen reader in action, add an `audio` element after the `p` element. Include the `controls` attribute. Then place a `source` element inside the `audio` tags with the `src` attribute set to `https://s3.amazonaws.com/freecodecamp/screen-reader.mp3` and `type` attribute set to `"audio/mpeg"`. </del> <ins> Time to take a break from Camper Cat and meet fellow camper Zersiax (@zersiax), a champion of accessibility and a screen reader user. To hear a clip of his screen reader in action, add an `audio` element after the `p` element. Include the `controls` attribute. Then place a `source` element inside the `audio` tags with the `src` attribute set to `https://cdn.freecodecamp.org/curriculum/applied-accessibility/screen-reader.mp3` and `type` attribute set to `"audio/mpeg"`. </ins> **Note:** The audio clip may sound fast and be difficult to understand, but that is a normal speed for screen reader users.

```js assert( $('source').attr('src') === <del> 'https://s3.amazonaws.com/freecodecamp/screen-reader.mp3' </del> <ins> 'https://cdn.freecodecamp.org/curriculum/applied-accessibility/screen-reader.mp3' </ins> ); ```

<main> <p>A sound clip of Zersiax's screen reader in action.</p> <audio controls> <del> <source src="https://s3.amazonaws.com/freecodecamp/screen-reader.mp3" type="audio/mpeg"/> </del> <ins> <source src="https://cdn.freecodecamp.org/curriculum/applied-accessibility/screen-reader.mp3" type="audio/mpeg"/> </ins> </audio> </main> </body>

} </style> <header> <del> <img src="https://freecodecamp.s3.amazonaws.com/quincy-twitter-photo.jpg" alt="Quincy Larson's profile picture" class="profile-thumbnail"> </del> <ins> <img src="https://cdn.freecodecamp.org/curriculum/legacy-css-flexbox/quincy-twitter-photo.jpg" alt="Quincy Larson's profile picture" class="profile-thumbnail"> </ins> <div class="profile-name"> <h3>Quincy Larson</h3> <h4>@ossia</h4>
Change:
In the JS Beta project to build a pyramid gen., steps 82 and 83 both show an if statement block in the code editor that is not correctly indented within the while loop it is in. This may confuse new learners if they are relying on the indents to understand the logic and should be fixed to keep the presented code in acceptable shape for learners. N/A the if statement should be indented within the while loops block ! ! N/A No response

Discussion:
--fcc-editable-region-- --fcc-editable-region-- <del> if (done === count) { </del> <ins> if (done === count) { </ins> continueLoop = false; } }

while (continueLoop) { done++; rows.push(padRow(done, count)); <del> if (done === count) { </del> <ins> if (done === count) { </ins> continueLoop = false; } }

while (done !== count) { done++; rows.push(padRow(done, count)); <del> if (done === count) { </del> <ins> if (done === count) { </ins> continueLoop = false; } }
Change:
When invalid code is submitted to step 4 of the gradebook app review project, a message is shown which has a typo. "Your studentMsg function should return a the correct ..." Please remove the just click submit without typing anything to trigger the error message display pls correct the typo No response N/A No response
Thanks for opening this issue. This looks something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: List of files: Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.
Hi I am intrested to work on this, Can I start?
iam interested to work on this issue
Hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Hi, can I take up this issue? I am new to open source so looking to start with some small contributions.
We do not assign issues except to long time contributors and members. please read through this note here on contributing thanks
I have created a PR for the issue. Waiting for your feedback. Thank you!

Discussion:
assert.strictEqual(studentMsg([56, 23, 89, 42, 75, 11, 68, 34, 91, 19], 100), "Class average: 50.8. Your grade: A++. You passed the course."); ``` <del> Your `studentMsg` function should return a the correct message based on the student's score and the class average. </del> <ins> Your `studentMsg` function should return the correct message based on the student's score and the class average. </ins> ```js assert.strictEqual(studentMsg([33, 44, 55, 66, 77, 88, 99, 100], 92), "Class average: 70.25. Your grade: A. You passed the course.");
Change:
The instructions in step 18 of the js beta to build a calorie counter says: Start by declaring a cleanInputString function that takes a str parameter. However if you misspell the function's name you receive this hint: You should declare a cleanInputString variable. I believe this message should be changed to say "function" instead of "variable" the hint should refer to a not a No response N/A No response
Similar issue was seen in step 28 where the hint said: You should declare an isInvalidInput variable. while the instructions were again to create a function: Start by creating a function called isInvalidInput – it should take a single str parameter. And again, similar issue in step38 hint says: You should declare an addEntry variable. while the step is asking for a function
From Pyramid Generator A function is a block of code that can be reused throughout your application. Functions are declared with the following syntax:
Step 42 mentioned in the previous comment should be fixed. It should say: “The function declaration tells JavaScript that the name identifier is going to be a function.” (Replacing ‘variable’ with ‘identifier’ and ‘keyword’ with ‘declaration’) My understanding is that a function is actually an object in js but I would be surprised if a js course starts telling learners to “declare an object called myFunction” instead of saying “declare a function called myFunction”. Edited…
Step 42 should be a separate issue IMO. It shouldn't be lumped in with the corrections being made to the calorie counter steps
Here the updates for this issue remove this hint and test js assert.isDefined(cleanInputString); the current second hint and test should say this instead js assert.isFunction(cleanInputString); remove this hint and test js assert.isDefined(isInvalidInput) remove this hint and test js assert.isDefined(addEntry); udpate the second hint to the following js assert.isFunction(addEntry); `

Discussion:
# --hints-- <del> You should declare a `cleanInputString` variable. ```js assert.isDefined(cleanInputString); ``` Your `cleanInputString` variable should be a function. </del> <ins> You should declare a `cleanInputString` function. </ins> ```js assert.isFunction(cleanInputString);

# --hints-- <del> You should declare an `isInvalidInput` variable. ```js assert.isDefined(isInvalidInput) ``` </del> `isInvalidInput` should be a function. ```js

# --hints-- <del> You should declare an `addEntry` variable. ```js assert.isDefined(addEntry); ``` Your `addEntry` variable should be a function. </del> <ins> You should declare an `addEntry` function. </ins> ```js assert.isFunction(addEntry);
Change:
The step asks for a list comprehension, but the following code is accepted as a solution (with parenthesis): I believe the step must only accept the list comprehension with square brackets []: No response Device: Laptop OS: Windows 11 Browser: Brave Version: 1.64.116 No response
Thank you for reporting this issue. I don't think this is a huge problem, since the code works well with a generator expression too. However, since the instructions are asking for a list comprehension, it makes sense to fix the test. The helpers do not discriminate between the various comprehension types, so I'd import the module and check if the value of the variable is an instance of . This additional check should take place within .

Discussion:
You should assign a list comprehension that iterates over `row` to the `row_str` variable. ```js <del> ({ test: () => assert(runPython(`_Node(_code).find_class("Board").find_function("__str__").find_for_loops()[0].find_bodies()[0].find_variable("row_str").find_comp_iters()[0].is_equivalent("row")`)) }) </del> <ins> ({ test: () => (runPython(` import ast var = _Node(_code).find_class("Board").find_function("__str__").find_for_loops()[0].find_bodies()[0].find_variable("row_str") assert var.find_comp_iters()[0].is_equivalent("row"), "You should iterate over row" assert isinstance(var.tree.value, ast.ListComp), "It's not a list comprehension" `)) }) </ins> ``` The list comprehension assigned to the `row_str` variable should use `i` as iteration variable.
Change:
This sentence seems to be confusing campers on the forum Let's go with this updated description here see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the Happy coding
Hi, I have created this PR as a first time contributor (no PRs merged yet) to this codebase. Hope the changes are up to expected requirements. Thanks!

Discussion:
Variable naming follows specific rules: names can include letters, numbers, dollar signs, and underscores, but cannot contain spaces and must not begin with a number. <del> Declare a `character` variable in your code. </del> <ins> Use the `let` keyword to declare a variable called `character`. </ins> _Note_: It is common practice to end statements in JavaScript with a semicolon. `;`
Change:
The contains a lot of unused CSS rules, which should be cleaned up. The unused rules/classes are: .challenge-success-modal .btn-lg .btn-cta-big .completion-modal-body .completion-message .completion-challenge-nameRemove the above classes from Test and confirm that the modal displays properly after the changes
Hi, I can help with this If you would like to assign it to me.

Discussion:
} @media screen and (max-width: 991px) { <del> .challenge-success-modal .btn-lg { font-size: 1rem; } .challenge-success-modal .btn-cta-big { max-width: 100%; font-size: 1rem; } .completion-modal-body { min-height: 340px; } </del> .progress-bar-wrap, .progress-bar-background { height: 10px;

font-size: 1.2rem; } <del> .completion-challenge-name, </del> .completion-block-name { font-size: 1rem; }