Skip to main content

Release 0.1

The past few weeks I have been looking at the differences between declaring variables for JavaScript.

Var, Const, Let are the variable declarations that are widely used, each one having a specific purpose. New developers mostly use 'var,' as it is the first technique taught.

However, 'var' declarations is no longer widely used by JavaScript developers. The modern practice of variable declarations is with 'const' and 'let'.

Before starting anything, I had to first understand the difference between the three:
  • Const declarations means the values cannot be changed/reassigned. 
  • Let declarations should be used when declaring local variables in a block scope. 
  • Var declarations hold the weakest signal which can be used outside of a block, and can be reassigned.
For Release 0.1, we were asked to look into a library called 'filer.js.' The task was to go through one of the test files, and change the old-fashioned 'var' declarations to the modern 'const/let' declarations. The task was to fork the GitHub library and create a new branch to make changes to. After that, we were required to create a pull request to make the changes to the 'filer.js' master branch

My Issue: https://github.com/filerjs/filer/issues/728
My Pull-Request: https://github.com/filerjs/filer/pull/730

The file I explored was 'filer/tests/spec/fs.shell.spec.js.' The file contained only 'var' declarations, and needed to be updated to the modern format of 'const/let.' The first step I took was to fork the library to my own GitHub account. After that, I opened Terminal and pulled the repository to my local machine. I created a new branch in the new forked repository, and was ready to make changes. Through Visual Studio Code, I edited the 'fs.shell.spec' file and added it to the git files ready to be pushed. After pushing the files to the repository, I created a pull request to the original filer.js library.

Upon completing my own task, I went into all of the pull requests made by my peers. Most of my peers correctly changed the variable declarations, but I found one that had some issues.

Reviewed Pull-Request: https://github.com/filerjs/filer/pull/729

This pull request declared all of the variables with 'let' which according to the articles I read, was wrong. The variables should have all used 'const,' as the variables did not change values. Some examples of the code:


Upon finding these errors, I commented a message to let my peer know to change it.



Other peers have already been reviewed, and this issue was the only one left that had errors.



Comments

Popular posts from this blog

Lab 8

This lab is the final lab regarding this course, so I chose two issues to work on for the span of the last two Releases (0.3, 0.4). The first issue I found was an RPG game that needed to implement global stats that track the players total player deaths, monster kills by type and top stats. The language used is JavaScript Issue 1 (0.3):  https://github.com/ASteinheiser/react-rpg.com/issues/14 The second issue I found was for a rock-paper-scissors game, that needs to implement a Refresh button that resets all the win/loss/ties back to 0. Issue 2 (0.4):  https://github.com/golemheavy/RPS-Multiplayer/issues/14

Release 0.2 - Lab 4

Issue: https://github.com/RabanserD/rabanserd.github.io/issues/75 This week I managed to find an issue that required a pull request template. The required technology for this issue was github and '.md' for the documentation.

Release 0.2 - Lab 7

Repository: https://github.com/Jesse989/basic-fantasy-rpg Issue: There was no specific issue for this week that I found. The problem was that I went through the documentation for this repository, and found a spelling mistake. The spelling mistake I found was 'stats' instead of status. The owner of the repository had thanked me for going through the documentation, and accepted my pull request to update the spelling mistake.