this year i participated again in the hacktoberfest, and itβs been fun!
i had half a month to actually complete the challenge (to have 4 pull requests merged and approved), since i was on vacation π π.
contributions
clio-lang
a simple renaming of a method got me this merged.
i noticed this naming could be improved, as star and transform are two separate names and should be separated by _.
e.g.
- startransform: function(node) {
+ star_transform: function(node) {
problem-solving-javascript
added tests for bottom view binary tree
e.g.
+ const BinaryTree = require("../index");
+ const bottomView = require('.');
+
+ describe('Bottom View Binary Tree', () => {
+ let btree
+
+ beforeEach(() => {
+ btree = new BinaryTree([1, 2, 3, 4, 5, 6]);
+ });
+
+ it('Should determine the bottom view of a binary tree', () => {
+ expect(bottomView(btree)).toEqual([6, 2, 3, 4]);
+ });
+ it('Should handle null binary tree', () => {
+ expect(bottomView(null)).toEqual([]);
+ });
+ });
m.args
this should help with unifying the readme to other m.projects
m.static
this pr updates the badges in the readme
another year, another hacktoberfest
did you participate this year?