jiebi/CodeConvo huggingface dataset:
Change:
The function used in the code, named : Instructions mention function: "Because returns a function, your function ultimately returns an array of function references..." Instruction should mention function, not function. No response Device: Laptop OS: Windows 10 Browser: Chrome 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.
can you assign me this issue?
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.

Discussion:
# --description-- <del> Because `elemValue` returns a function, your `addChars` function ultimately returns an array of function references. You want the `.map()` method to run the inner function of your `elemValue` function, which means you need to call `elemValue` instead of reference it. Pass `num` as the argument to your `elemValue` function. </del> <ins> Because `elemValue` returns a function, your `addCharacters` function ultimately returns an array of function references. You want the `.map()` method to run the inner function of your `elemValue` function, which means you need to call `elemValue` instead of reference it. Pass `num` as the argument to your `elemValue` function. </ins> <!-- TODO: Explain further? -->
Change:
A wrong solution is accepted at coding-interview-prep/data-structures/remove-an-element-from-a-max-heap challenge The test case for The remove method should remove the greatest element from the max heap while maintaining the max heap property should not pass. The next example shows that, the function does not maintain the max heap properly: Device: [Laptop] OS: [ Windows 10] Browser: [Chrome] Version: [121.0.6167.86]
OK, I have make it.

Discussion:
```js function isHeap(arr, i, n) { <del> if (i >= (n - 1) / 2) { return true; } if ( arr[i] >= arr[2 * i + 1] && arr[i] >= arr[2 * i + 2] && isHeap(arr, 2 * i + 1, n) && isHeap(arr, 2 * i + 2, n) ) { return true; } return false; </del> <ins> if( arr[i] < arr[2 * i + 1] || arr[i] < arr[2 * i + 2] ){ return false; } if (i > (n - 1) / 2) { return true; } if (isHeap(arr, 2 * i + 1, n) && isHeap(arr, 2 * i + 2, n)) { return true; } return false; </ins> } assert( (function () { let test = false; if (typeof MaxHeap !== 'undefined') { test = new MaxHeap(); } else { return false; } <del> let max = Infinity; const [result, vals] = [[], [2, 15, 3, 7, 12, 7, 10, 90]]; vals.forEach((val) => test.insert(val)); for (let i = 0; i < vals.length; i++) { const curHeap = test.print(); const arr = curHeap[0] === null ? curHeap.slice(1) : curHeap; if (!isHeap(arr, 0, arr.length - 1)) { return false; </del> <ins> let max = Infinity; const [result, vals] = [[], [9, 3, 5, 2, 15, 3, 7, 12, 7, 10, 90]]; vals.forEach((val) => test.insert(val)); for (let i = 0; i < vals.length; i++) { const curHeap = test.print(); const arr = curHeap[0] === null ? curHeap.slice(1) : curHeap; if (!isHeap(arr, 0, arr.length - 1)) { return false; } const removed = test.remove(); if (!vals.includes(removed)) return false; if (removed > max) return false max = removed; result.push(removed); </ins> } <del> const removed = test.remove(); if (!vals.includes(removed)) return false; if (removed > max) return false max = removed; result.push(removed); } for (let i = 0; i < vals.length; i++) { if (!result.includes(vals[i])) { return false; } } return true </del> <ins> for (let i = 0; i < vals.length; i++) { if (!result.includes(vals[i])) { return false; } } return true; </ins> })() ); ```
Change:
I noticed a typo in the instructions for Step 11 for The typo is in the line I think the word should be I know it's a bit pedantic, but this is so... N/A N/A No response Device: HP Laptop OS: Windows 10 Browser:Firefox Version: 122.0 No response

Discussion:
lambda x: expr ``` <del> In the example above, `x` is a parameter to be use in the expression `expr`. Create a `test` variable and assign it a lambda function that takes an `x` parameter and returns `x * 2`. </del> <ins> In the example above, `x` is a parameter to be used in the expression `expr`. Create a `test` variable and assign it a lambda function that takes an `x` parameter and returns `x * 2`. </ins> # --hints--
Change:
We still have a lot of confusion on the forum about step 41 I think adding a code example of what they need to do will help. Here is my proposed update to the step. js button.onclick = myFunction; ` 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
Just a note - that will be step 40 currently
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:
`button1` represents your first `button` element. These elements have a special property called `onclick`, which you can use to determine what happens when someone clicks that button. <del> You can access properties in JavaScript a couple of different ways. The first is with <dfn>dot notation</dfn>. Accessing the `onclick` property of a button would look like: </del> <ins> You can access properties in JavaScript a couple of different ways. The first is with <dfn>dot notation</dfn>. Here is an example of using dot notation to set the `onclick` property of a button to a function reference. </ins> ```js <del> button.onclick </del> <ins> button.onclick = myFunction; </ins> ``` <ins> In this example, `button` is the button element, and `myFunction` is a reference to a function. When the button is clicked, `myFunction` will be called. </ins> Use dot notation to set the `onclick` property of your `button1` to the function reference of `goStore`. Note that `button1` is already declared, so you don't need to use `let` or `const`. # --hints--
Change:
There is a lot of confusion on the forum about step 117 in the rpg game The goal of this step is to teach campers how to work with the style property. Updating the description and code example will clear up what is meant to happen in this step. js const paragraph = document.querySelector('p'); = 'block'; ` 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

Discussion:
# --description-- <del> The HTML element that shows the monster's stats has been hidden with CSS. Display the `monsterStats` element by updating the `display` property of the `style` property to `block`. For example, updating the `first` property of the `name` property of `user` would look like: </del> <ins> By default, the HTML element that shows the monster's stats has been hidden with CSS. When the player clicks the "Fight dragon" button, the monster's stats should be displayed. You can accomplish this by using the <dfn>style</dfn> and <dfn>display</dfn> properties on the `monsterStats` element. The `style` property is used to access the inline style of an element and the `display` property is used to set the visibility of an element. Here is an example of how to update the display for a paragraph element: </ins> ```js <del> user.name.first = "Naomi"; </del> <ins> const paragraph = document.querySelector('p'); paragraph.style.display = 'block'; </ins> ``` <ins> Display the `monsterStats` element by updating the `display` property of the `style` property to `block`. </ins> # --hints-- You should use dot notation to access the `style` property of `monsterStats`.
Change:
Throughout the curriculum, we often have campers name variables that represent booleans with the prefix of or which follow standard convention. But we never really explain that to them. I think that step 17 would be a good place do that. js let isRunning = true; let hasCompleted = false; ` 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

Discussion:
# --description-- <ins> In programming, it is standard practice to prefix a variable with `is` or `has` to indicate that the variable is a boolean. ```js let isRunning = true; let hasCompleted = false; ``` </ins> Declare an `isError` variable and set it to `false`, but use `let` so you can reassign it later. # --hints--
Change:
We introduce the method in step 79, but do not provide a code example. For consistency, we should provide code examples for these methods when they are first introduced. js const firstName = 'JESSICA'; (firstName.toLowerCase()); // Output: jessica ` 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
Hello :) newbie here, I am going to tackle this task. PR will be published soon.

Discussion:
# --description-- <del> Your `output.innerHTML` string will need a `span` element. Create that, and give it a `class` attribute set to the `surplusOrDeficit` variable, but lowercased. </del> <ins> When you need to lower case a string, you can use the <dfn>toLowerCase()</dfn> method. This method returns the calling string value converted to lower case. </ins> <del> Strings have a `.toLowerCase()` method that can help you with this. Do not give your `span` any text yet. </del> <ins> ```js const firstName = 'JESSICA'; console.log(firstName.toLowerCase()); // Output: jessica ``` Your `output.innerHTML` string will need a `span` element. Create that, and give it a `class` attribute set to the `surplusOrDeficit` variable. Your `surplusOrDeficit` variable should be converted to lower case using the `toLowerCase()` method. Do not give your `span` any text yet. </ins> # --hints--
Change:
We should add a code example for the remove method in step 85. js const paragraphElement = document.getElementById('paragraph'); ('hide'); ` 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
Can we make this a issue?
Normally this would be a first timers only issue. But I already opened a tons of those earlier, that are still up for grabs. So I decided to make this one help wanted, to open it up to more people. Plus we still have those pretty earlier on that made one contribution but are still looking for easier issues to keep contributing.

Discussion:
Finally, you need to make the `#output` element visible so the user can see your text. Your `output` variable is an Element, which has a `classList` property. This property has a `.remove()` method, which accepts a string representing the class to remove from the element. <del> Use the `.remove()` method of the `output` variable's `classList` property to remove the `hide` class. Don't forget to place the word `hide` inside quotes. </del> <ins> ```js const paragraphElement = document.getElementById('paragraph'); paragraphElement.classList.remove('hide'); ``` Use the `.remove()` method of the `output` variable's `classList` property to remove the `hide` class. Don't forget to place the word `hide` inside quotes. </ins> # --hints--
Change:
following task ask for : reading this line suggests/ indicates to me writing an "if" statement inside of the array although that could be my understanding of the sentence my suggestion would be to update the sentence to the following No response Device: [macBook pro] OS: [Ventura 13.4.1 ] Browser: [chrome ] Version: [121.0] No response
Hi ! Thank you for creating this issue. I think the re wording makes sense. The only thing, I would change would be the first part of that sentence. We should mention explicitly that they are adding to the array. I will go ahead and open this up for contribution for
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
I updated the instructions of step 78 and committed the changes. Please have a look at that Thank you
I am sorry but since I am new to open-source contributions, I accidentally closed the previous PR. I am making a new Pull Request please have a look at that.

Discussion:
Inside the callback function, create a new `const` variable called `collisionDetectionRules` and assign it an empty array. <del> Inside that array, add a boolean expression that checks if the player's `y` position plus the player's height is less than or equal to the platform's `y` position. </del> <ins> Inside that array, add a boolean expression that checks whether the player's `y` position plus the player's height is less than or equal to the platform's `y` position. </ins> # --hints--

assert.match(code, /consts+collisionDetectionRuless*=s*[s*(?:[^]]*s*)*s*]s*;?/); ``` <del> You should have a boolean expression that checks if the player's `y` position plus the player's height is less than or equal to the platform's `y` position. </del> <ins> You should have a boolean expression that checks whether the player's `y` position plus the player's height is less than or equal to the platform's `y` position. </ins> ```js assert.match(code, /consts+collisionDetectionRuless*=s*[s*player.position.ys*+s*player.heights*<=s*platform.position.ys*,?s*]s*;?/);
Change:
The instruction are not clearing, it take times to know what it is saying The heading should read <img width="832" alt="Screenshot 2024-02-05 at 21 30 18" src=""- Device: Laptop OS: Mac Browser: Chrome No response
It looks like the steps have been updated on the main branch but these changes are not live on the site. So the step you are referring will be step 110 here.
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> Following the same pattern, use that code in the `fightBeast` and `fightDragon` functions. Remember that `beast` is at index `1` and `dragon` is at index `2`. Also, remove the `console.log` call from your `fightDragon` function. </del> <ins> Following the same pattern as the `fightSlime` function, use that code in the `fightBeast` and `fightDragon` functions. Remember that `beast` is at index `1` and `dragon` is at index `2`. Also, remove the `console.log` call from your `fightDragon` function. </ins> # --hints--
Change:
We have a spelling error for step 3 of the stats calculator project. The word separating is spelled wrong in the description. 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

Discussion:
The `.split()` method takes a string and splits it into an array of strings. You can pass it a string of characters or a RegEx to use as a separator. For example, `string.split(",")` would split the string at each comma and return an array of strings. <del> Use the `/,s*/g` regex to split the `value` string by commas. You can tweak it based on the number of spaces seperating your values. Store the array in an `array` variable. </del> <ins> Use the `/,s*/g` regex to split the `value` string by commas. You can tweak it based on the number of spaces separating your values. Store the array in an `array` variable. </ins> # --hints--
Change:
The font family for the 'trending guides' and 'mobile app' titles in the footer should be 'Lato', sans-serif and the .footer-desc-col in the footer should be aligned with the trending find the related selector for the footer-desc-col and add the following style align-self: start; Also, find the related selector to the footer titles and add the following style: font-family: var(--font-family-sans-serif); <img width="679" alt="Screenshot 2024-02-07 at 3 26 07 PM" src=""
Hi, I'd like to take this issue please
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.

Discussion:
.site-footer .col-header, .trending-guides .col-header { text-align: center; <ins> font-family: var(--font-family-sans-serif); </ins> font-weight: 700; font-size: 16px; padding: 0 15px 15px;

.footer-desc-col { padding-inline: 15px; margin-bottom: 30px; <del> align-self: center; </del> <ins> align-self: start; </ins> } .footer-desc-col a {
Change:
The instruction are not clear, right now it says Edit: This is actually step 120 in the main branch Set monsterHealth to monsterHealth minus the power of the player's current weapon. Remember you have the currentWeapon which is not initialized variable and the power property on weapons array. No response Device: Laptop OS: macOS Browser: Chrome I think it should be more detail about what and where is currentWeapon, power property and how to access it like in previous.
It looks like currentWeapon is initalized with the value of 0 here Then it is updated over time throughout the project. I think for that second sentence it should be clear about syntax on how to access those values since beginners are still struggling with that. Being more explicitly will help with confusion on the forum posts. Let's go with this change.
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> Set `monsterHealth` to `monsterHealth` minus the power of the player's current weapon. Remember you have the `currentWeapon` variable and the `power` property. </del> <ins> Set `monsterHealth` to `monsterHealth` minus the power of the player's current weapon. Remember that you can access the power of the player's current weapon using `weapons[currentWeapon].power`. </ins> # --hints--
Change:
This challenge does not involve editing at all, yet it is discussed in the hint text. The text should be like "Your function should return an arrow function which has a character2 parameter." so it is clear what needs edited. ! Device: Desktop OS: Windows 10 Browser: Firefox Version: 122.0.1 No response
55 "Kolade Chris"< &gt;; 2024/2/12 20:48 "freeCodeCamp/freeCodeCamp"< &gt;; "Subscribed"< &gt;; Re: [freeCodeCamp/freeCodeCamp] Confusing hint text in Learnfunctional programming step 46 (Issue ) Closed as completed via . — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID:

Discussion:
assert.notMatch(code, /consts+evalFormulas*=s*(s*xs*,s*cellss*)s*=>s*{s*consts+idToTexts*=s*(?s*ids*)?s*=>s*cells.find(s*(?s*cells*)?s*=>s*(?:cell.ids*===s*id|ids*===s*cell.id)s*).value;?s*consts+rangeRegexs*=s*/([A-J])([1-9][0-9]?):([A-J])([1-9][0-9]?)/(gi|ig);?s*consts+rangeFromStrings*=s*(s*num1s*,s*num2s*)s*=>s*range(s*parseInt(s*num1s*)s*,s*parseInt(s*num2s*)s*);?s*consts+elemValues*=s*(?s*nums*)?s*=>s*{s*consts+inners*=s*(?s*characters*)?s*=>s*{s*returns+idToText(s*characters*+s*nums*);?s*};?s*returns+inner;?s*}s*consts+addCharacterss*=s*((s*character1s*)|characters1)s*=>s*{/); ``` <del> Your `elemValue` function should return an arrow function which has a `character2` parameter. </del> <ins> Your `addCharacters` function should return an arrow function which has a `character2` parameter. </ins> ```js assert.match(code, /consts+evalFormulas*=s*(s*xs*,s*cellss*)s*=>s*{s*consts+idToTexts*=s*(?s*ids*)?s*=>s*cells.find(s*(?s*cells*)?s*=>s*(?:cell.ids*===s*id|ids*===s*cell.id)s*).value;?s*consts+rangeRegexs*=s*/([A-J])([1-9][0-9]?):([A-J])([1-9][0-9]?)/(gi|ig);?s*consts+rangeFromStrings*=s*(s*num1s*,s*num2s*)s*=>s*range(s*parseInt(s*num1s*)s*,s*parseInt(s*num2s*)s*);?s*consts+elemValues*=s*(?s*nums*)?s*=>s*{s*consts+inners*=s*(?s*characters*)?s*=>s*{s*returns+idToText(s*characters*+s*nums*);?s*};?s*returns+inner;?s*}s*consts+addCharacterss*=s*((s*character1s*)|character1)s*=>s*((s*character2s*)|character2)s*=>/);
Change:
A camper on the forum was unable to pass with the following code: The tests are only checking for in the loop declaration That should be fixed to allow for or or declarations should be valid 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:
```js const clearForm = code.split("function clearForm()")[1]; <del> assert.match(clearForm, /fors*(s*consts+containers+ofs+inputContainerss*)/); </del> <ins> assert.match(clearForm, /fors*(s*(const|let)s+containers+ofs+inputContainerss*)/); </ins> ``` Your `for...of` loop should set the `innerHTML` property of `container` to an empty string.