jiebi/CodeConvo huggingface dataset:
Change:
"assert.deepEqual(palindrome("not a palindrome"), false);", "assert.deepEqual(palindrome("A man, a plan, a canal. Panama"), true);", "assert.deepEqual(palindrome("never odd or even"), true);", <del> "assert.deepEqual(palindrome("nope"), false);" </del> <ins> "assert.deepEqual(palindrome("nope"), false);", "assert.deepEqual(palindrome("almostomla"), false);" </ins> ], "challengeSeed": [ "function palindrome(str) {",

Discussion:
There is a bug in the test, I wrote the code to check the string palindrome or not like this. function palindrome(str) { // Good luck! var i, j; for (i = 0,j = ( -1) ; i < j; i++,j--){ if(str[i].toLowerCase() !== str[j].toLowerCase()) return false; else return true; } } If you see this program it looks correct. yes this is correct to check the string palindrome or not. but this is wrong in returning. it wont loop entire string, With this code all the tests are passing, but it should not pass any test, at least 2 test should fail.
Pleese close this and take an extra look at your if-statement. The #help channel on Slack is a great resource if you can't figure it out, or you can @ me on there and I'll help you out.
Hi I know my program is wrong. That's what I am mentioning with this program all test cases are passing, and it allowed me to go to next exercise. Regards Namitha On Saturday, June 13, 2015, Anton Strömkvist wrote:
Oh, sorry, my bad! Should be fixed with
Change:
| Username not found label.negative-10.btn.btn-primary.btn-block#submitButton i.fa.fa-play <del> | &nbsp; Run code (ctrl + enter) </del> <ins> | &nbsp; Run tests (ctrl + enter) </ins> .button-spacer .btn-group.input-group.btn-group-justified label.btn.btn-success#trigger-reset-modal

Discussion:
I noticed on this challenge that the button to run the tests still runs code ! I think this should say run tests
Change:
# --instructions-- <del> Within `tests/2_functional-tests.js`, in the `'Submit the surname "Vespucci" in the HTML form'` test (`// #5`), automate the following: </del> <ins> Within `tests/2_functional-tests.js`, in the `'Submit the surname "Vespucci" in the HTML form'` test (`// #6`), automate the following: </ins> 1.  Fill in the form with the surname `Vespucci` 2.  Press the submit button

Discussion:
Two of the typos are minor but one could be confusing In the first paragraph of the instructions (see below), there is a reference to The actual test number to change is and not , so it should be changed to reflect this. n/a 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_
Hey, I created pull request for this issue.
Change:
min-width: 150px; } <del> .media { word-break: break-all; } </del> /* Timeline pagination */ .timeline-pagination_list { list-style: none;

Discussion:
Affected page Any profile with a portfolio item: To Reproduce 1) Go to above portfolio link 2) Scroll to first and only portfolio item 3) Be shocked and horrified at word-wrapping Expected behaviour There should be no word wrapping breaking the word. I suggest something be to the CSS like: Screenshots ! Reported on forum: An issue with the help wanted or first timers only label is open 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
Hi I had a look at this. It appears that this line is creating the mess. Wouldn't it be better to just remove that rule? I've tested it through dev tools and it works. Also checked the source and it appears that is only referenced in , so there are probably no unwanted side effects. Unfortunately, I can't access the profile page on my local setup to test. On a sidenote, how could I setup my local instance so that I can access the profile page? Anyway, what do you think?
Potentially. But, as you say, only if there are no side-effects. It is in this section of the . If you want to make the change, and open a PR, I think that makes sense. We will QA it, and, if anyone notices the change breaking something else, then we can work from there. Thanks, for digging deeper into this.
Thanks! Will send a PR, tomorrow!
Change:
assert(isValidSolution_(solve24(testCases_[1]))); ``` <del> `solve24("6789")` should return `(6*8)/(9-7)`. `(8*6)/(9-7)`, or a similar valid string </del> <ins> `solve24("6789")` should return `(6*8)/(9-7)`, `(8*6)/(9-7)`, or a similar valid string </ins> ```js assert(isValidSolution_(solve24(testCases_[2])));

Discussion:
In both the sentences on this line the word should be surrounded with backticks: In this line there is a period (full-stop) where there should be a comma. 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: List of files: 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.
Hello, I want to work on this.
Heyy, is this assigned to anyone?
I will work on this.
Hey! I would like to work on this issue.
Hi I have fixed the issue and you can find the PR here
Is this issue available or has it been resolved?
Change:
```html <style> <ins> body { min-height: 150vh; } </ins> #navbar { position: fixed; top: 0;

Discussion:
Describe your problem and how to reproduce it: We were in the middle of a meetup that was oriented at beginners, and a camper that was doing this challenge asked me: what does do? I feel like it works exactly as I showed them that it kept the element in place when the page scrolled. The description says it One key difference between the and positions is that an element with a fixed position won't move when the user scrolls. However, it doesn't provide a visible example where the user can scroll and see that for themselves. Add text and styles to invite campers to scroll the content and see how differs from . Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Google Chrome Browser Version: Version 77.0.3865.90 Operating System: Elementary OS Loki 0.4.1 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): ! As you can see, the content isn't scrollable
Change:
cursor: pointer; display: flex; font-weight: normal; <del> border: 2px solid var(--secondary-background); </del> <ins> border-left: 4px solid var(--tertiary-background); border-right: 4px solid var(--tertiary-background); border-top: 2px solid var(--tertiary-background); border-bottom: 2px solid var(--tertiary-background); } .video-quiz-option-label:first-child { border-top: 4px solid var(--tertiary-background); } .video-quiz-option-label:last-child { border-bottom: 4px solid var(--tertiary-background); </ins> } .video-quiz-input-hidden {

Discussion:
<img width="492" alt="Screen Shot 2020-05-28 at 1 29 28 PM" src="https://user-"<img width="511" alt="Screen Shot 2020-05-28 at 1 28 37 PM" src="https://user-"<hr<img width="518" alt="Screen Shot 2020-05-28 at 1 53 35 PM" src="https://user-"
thanks for catching the code blocks in the answer section looking off. Didn't notice it at first -- maybe it required a refresh? But yeah, looks off with the darker background. Tested this locally and it seems like we can just remove the language (py or python) from the triple backticks: ! I also think it would be better to have all the code in markdown. It's a bit finicky in conjunction with YAML, but makes it much easier to control the formatting/spacing than tags.
How about something like this: ! this is with a background of and using to enable highlighting. I'm not sure what the best background is. This one looks a little better than white and is a bit better from a contrast perspective.
I would like to see the same background color for answers like in Kris's photo, but the code highlighting, like in Oliver's. I think the only way to achieve this is to change some styles after the fact in the css.
Here are some options: ! the first two being white and prism's default (#). The issue being they look progressively better, but get progressively harder to read. in particular starts to fade into the background. That's what got me thinking about this , so I think it's fine to go with the prettiest version as long as we address the a11y issues.
what about changing the whole box to white... <img width="446" alt="Screen Shot 2020-05-29 at 3 41 42 PM" src="https://user-"Kinda solves both issues
Makes sense to me. They might look a little better with a dark border, but either way, I think the white background is the way to go.
Change:
"tests": [], "type": "waypoint", "challengeType": 3, <del> "nameEs": "Crea una aplicación de votaciones" </del> <ins> "nameEs": "Desafios para aprender React", "descriptionEs": [] </ins> } ] }

Discussion:
[x] Learn D3 Challenges
what should we do with these ones? I guess they are not finished, so there's no point in translating that one line, or am I wrong?
Change:
); ``` <del> The state of `StatefulComponent` should be initialized with a property `name` set to a string. </del> <ins> The state of `StatefulComponent` should be initialized with a property `firstName` set to a string. </ins> ```js assert(

Discussion:
A user wondering why his solution passed. I then realized the 2nd test test is not working as intended. The user is supposed to assign a string to a property of . The user's solution assigns an uninitialized variable as a property. The purpose of the test is to make sure the user assigns a string value to a prop named , but the test still allows the user to pass as the prop's value and the test passes. Not sure why, but assigning an uninitialized property to ends up assigned an empty string to it. This of course passes the test requirement but defeats the purpose of the learning here. I suggest changed the required property name to something like , so if a user does something like this, all of the tests will fail. Test in question: The second test should fail because the property of should be undefinedbecause it was not initialized by the user's code. 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
After a bit of research, it appears that assigning as a property that is uninitialized is the same as writing: = { } Since the window has no name, it is just an empty string.
Change:
import MultifileEditor from './multifile-editor'; import DesktopLayout from './desktop-layout'; import MobileLayout from './mobile-layout'; <ins> import { mergeChallengeFiles } from './saved-challenges'; </ins> import './classic.css'; import '../components/test-frame.css';

Discussion:
Code in the doesn't seem to be evaluated in production for Cash Register project. (Edit)This appear to be related to the code saving / resetting. In development there isn't such issue. Including building production client and serving it. This is actually the only challenge from the new challenges that is using . Old challenges are working fine when using . Both when names start with ` might be somehow deciding factor, but there isn't any other new challenge to check that. After adding function to user code, error moves normally to the next used in test non-defined variable - and then . Steps to reproduce the behavior: (Be logged in, and have previously saved code for project). to solution. Example: tests. errors. Code present in the should be evaluated after the user code, and before running tests. (Edit) When logged out there's different error, or no errors at all(?!). This might be somehow related to saving / resetting code.
Thanks again for the detailed report, You were dead right about the issue: it's due to the fact that saved challenges don't include head or tail (aka and ) and we were just using them in place of challengeFiles (which does have that stuff).
I have marked this as a showstopper seeing as it is affecting camper's ability to complete a certificate project. It would be nice to get the fix in as soon as possible, so the forum isn't filled with support threads for this.
Change:
# --description-- <del> In the game of [golf](https://en.wikipedia.org/wiki/Golf) each hole has a `par` meaning the average number of `strokes` a golfer is expected to make in order to sink the ball in a hole to complete the play. Depending on how far above or below `par` your `strokes` are, there is a different nickname. </del> <ins> In the game of [golf](https://en.wikipedia.org/wiki/Golf), each hole has a `par`, meaning, the average number of `strokes` a golfer is expected to make in order to sink the ball in the hole to complete the play. Depending on how far above or below `par` your `strokes` are, there is a different nickname. </ins> Your function will be passed `par` and `strokes` arguments. Return the correct string according to this table which lists the strokes in order of priority; top (highest) to bottom (lowest):

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Describe your problem and how to reproduce it: Commas are missing in the paragraph, making it hard to read and understand the meaning behind the problem. It should be something like this: "In the game of golf, each hole has a par, meaning, the average number of strokes a golfer is expected to make in order to sink the ball in the hole to complete the play. Depending on how far above or below par your strokes are, there is a different nickname." Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Google Chrome Browser Version: Version 89.0.4389.90 Operating System: macOS Big Sur Version 11.2.3 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): <img width="712" alt="Screen Shot 2021-03-19 at 10 20 21 AM" src="https://user-"
Thanks for opening this issue. This looks something that can be fixed by "first time" code contributors to this repository. Along improving that part of text, it also should be considered, per , if external link to Wikipedia page about golf is necessary for completing challenge. 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 our chat room or the forum 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.
Change:
</FullWidthRow> <Spacer size='medium' /> <FullWidthRow> <del> <form id='form-quincy-email' onSubmit={handleSubmit}> </del> <ins> <form id='form-quincy-email' onSubmit={e => e.preventDefault()}> </ins> <ToggleButtonSetting action={t('settings.email.weekly')} flag={sendQuincyEmail}

Discussion:
Describe the bug When you try to make api call in the setting page, to change your subscription to Quincy newsletter, you will see a failed API call before the new one. To Reproduce Steps to reproduce the behavior: to 'Email Settings' in on unsubscribe from Quincy news-letter alert show before the success alert shown. Expected behavior Wait until the api fail before showing the alert
Change:
"assert.deepEqual(diff(["andesite", "grass", "dirt", "dead shrub"], ["andesite", "grass", "dirt", "dead shrub"]), [], 'message: <code>["andesite", "grass", "dirt", "dead shrub"], ["andesite", "grass", "dirt", "dead shrub"]</code> should return <code>[]</code>.');", "assert.deepEqual(diff([1, 2, 3, 5], [1, 2, 3, 4, 5]), [4], 'message: <code>[1, 2, 3, 5], [1, 2, 3, 4, 5]</code> should return <code>[4]</code>.');", "assert.includeMembers(diff([1, "calf", 3, "piglet"], [1, "calf", 3, 4]), ["piglet", 4], 'message: <code>[1, "calf", 3, "piglet"], [1, "calf", 3, 4]</code> should return <code>["piglet", 4]</code>.');", <del> "assert.deepEqual(diff([], ["snuffleupagus", "cookie monster", "elmo"]), ["snuffleupagus", "cookie monster", "elmo"], 'message: <code>[], ["snuffleupagus", "cookie monster", "elmo"]</code> should return <code>["snuffleupagus", "cookie monster", "elmo"]</code>.');" </del> <ins> "assert.deepEqual(diff([], ["snuffleupagus", "cookie monster", "elmo"]), ["snuffleupagus", "cookie monster", "elmo"], 'message: <code>[], ["snuffleupagus", "cookie monster", "elmo"]</code> should return <code>["snuffleupagus", "cookie monster", "elmo"]</code>.');", "assert.includeMembers(diff([1, "calf", 3, "piglet"], [7, "filly"]), [1, "calf", 3, "piglet", 7, "filly"], 'message: <code>[1, "calf", 3, "piglet"], [7, "filly"]</code> should return <code>[1, "calf", 3, "piglet", 7, "filly"]</code>.');" </ins> ], "MDNlinks": [ "Comparison Operators",

Discussion:
This should have all four elements in the final array but it does not. Have to beef up the validation on the tests.
Correct code for testing purposes.
It looks like this is for
Change:
Your `meta` element should have a `charset` attribute with the value `UTF-8`. ```js <del> assert.match(code, /<meta[sS]+?charset=('|"|`)UTF-81/) </del> <ins> assert.match(code, /<meta[sS]+?charset=('|"|`)UTF-81/i) </ins> ``` Your code should have a `title` element.

Discussion:
The value for is case-insensitive. Not sure if I missed any, but here are case-sensitive ones I found. Case sensitive curriculumchallengesenglish14-responsive-web-design-22learn-accessibility-by-building-a-quiz Case sensitive curriculumchallengesenglish14-responsive-web-design-22learn-css-animation-by-building-a-ferris-wheel Case sensitive curriculumchallengesenglish14-responsive-web-design-22learn-intermediate-css-by-building-a-picasso-painting Case sensitive curriculumchallengesenglish14-responsive-web-design-22learn-responsive-web-design-by-building-a-piano Case sensitive curriculumchallengesenglish14-responsive-web-design-22learn-the-css-box-model-by-building-a-rothko-painting a lot Both are valid The value for should be case insensitive for all challenges. 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]
Change:
'showMultifileProjectSolution' ); expect(getSolutionDisplayType(multifilePythonSolution)).toBe( <del> 'showMultifileProjectSolution' </del> <ins> 'showUserCode' </ins> ); }); it('should handle solutions with a single valid url', () => {

Discussion:
After submitting one of the new multifile Python cert projects, when you try to view the project on your settings page - it shows a blank page (links are to .dev) to a a solution to down to the project you just submitted -blank page Either show the python code running, or just remove the button for those projects. !
Change:
], "tests": [ "assert(secondToLastLetterOfLastName === 'c', 'secondToLastLetterOfLastName should be "c".');", <del> "assert(editor.getValue().match(/.length/g), 'You have to use .length to get the third last letter.');" </del> <ins> "assert(editor.getValue().match(/.length/g).length === 2, 'You have to use .length to get the third last letter.');" </ins> ], "challengeSeed": [ "var firstName = "Ada";",

Discussion:
Challenge has an issue. test asserting is used will always be true, as the example code in the editor uses it.
thanks for pointing this out! You're right. Actually that assert should check if there are minimum two 's in the code.
Change:
# --description-- <del> To calculate a root exponent, such as $sqrt[n]{x}$, you can use an inverted exponent $x^{1/n}$. </del> <ins> To calculate a root exponent, such as $sqrt[n]{x}$, you can use an inverted exponent $x^{1/n}$. JavaScript has a built-in `Math.pow()` function that can be used to calculate exponents. Here is the basic syntax for the `Math.pow()` function: ```js Math.pow(base, exponent); ``` Here is an example of how to calculate the square root of `4`: ```js const base = 4; const exponent = 0.5; // returns 2 Math.pow(base, exponent); ``` </ins> Declare a `standardDeviation` variable, and use the `Math.pow()` function to assign it the value of $variance^{1/2}$.

Discussion:
I struggled with the error message, see screenshot below. The error message asks challengers to use the variable, which I clearly did. So, it was not only unclear why the code didn't pass the test, but also misleading because the error message does not point to the real problem here: a second argument for (arg1, arg2) is missing. Since the instruction also doesn't provide a clear description of the syntax of (), I suggest that the syntax information of () be to the instruction. Albeit a minor issue, because anyone can look it up, I believe it would still make the instruction clearer. The error message should alert the challenger about a necessary second argument to pass in () if the variable is already passed in as the first argument. ! Device: Laptop OS: Windows 11 Browser: FireFox Version: 125.0.1 (64-bit) No response
For the new updated description, we could use this js (base, exponent) js const base = 4; const exponent = 0.5; // returns 2 (base, exponent); as for the hint text, it looks like the tests are broken up into two for In this case, I think it would be better to just have one test like this js (getStandardDeviation.toString(), /standardDeviations=sMath.pow(svariances,s1s/s2s)/);
Why step-51 in codebase is different also there is 58 steps showing when im running freecodecamp locally.But here there is only 55 steps.
The project is in beta which means the steps will be changing. So a recent update was made to the stats calculator project which means the step numbers were updated.
Change:
"nav": { "forum": "https://forum.freecodecamp.org/c/espanol/", "news": "https://freecodecamp.org/espanol/news/" <ins> }, "help": { "HTML-CSS": "Espanol/HTML-CSS", "JavaScript": "Espanol/JavaScript", "Python": "Espanol/Python", "Relational Databases": "Espanol/Relational Databases" </ins> } }

Discussion:
As we are now rolling out the curriculum in Chinese and Spanish, we need to think about how to get the button to work for the two languages. Currently, when the button is clicked, the user is taken to the English forum () where a pre-populated topic message (in English) is displayed for the user to fill in with relevant information. These topics are dynamically generated before the post is created on the forum, so we will just need to make use of the file for the content of the topics. Also, we will need to route to the correct forum url for Chinese and target the correct subforum for Spanish.
So the Chinese links work correctly, because the Chinese forum is hosted at a different domain (which is passed in to the environment variable). For Espanol, as it currently stands the category comes from the value returned by the .