Skip to content

add files to my branch#5

Open
GeFeSTTS wants to merge 4 commits intomasterfrom
domianych
Open

add files to my branch#5
GeFeSTTS wants to merge 4 commits intomasterfrom
domianych

Conversation

@GeFeSTTS
Copy link

Review please :)

skiffua
skiffua previously approved these changes May 16, 2019
Copy link

@skiffua skiffua left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like everything is correct:)

@skiffua
Copy link

skiffua commented May 16, 2019

Sorry, I closed your request by mistake

@GeFeSTTS GeFeSTTS requested review from Kindest13 and removed request for NadiiaChorna May 16, 2019 08:49
@GeFeSTTS GeFeSTTS requested a review from skiffua May 16, 2019 15:01
@GeFeSTTS GeFeSTTS force-pushed the domianych branch 3 times, most recently from 435c86e to ae45cbf Compare May 17, 2019 07:00
Kindest13
Kindest13 previously approved these changes May 17, 2019
skiffua
skiffua previously approved these changes May 17, 2019
@GeFeSTTS GeFeSTTS dismissed stale reviews from skiffua and Kindest13 via 53f987d May 17, 2019 08:47
skiffua
skiffua previously approved these changes May 17, 2019
@GeFeSTTS GeFeSTTS force-pushed the domianych branch 2 times, most recently from f110d6d to f803e7f Compare May 17, 2019 08:58
@skiffua skiffua self-requested a review May 17, 2019 09:03
@GeFeSTTS GeFeSTTS force-pushed the domianych branch 3 times, most recently from c30b057 to 03feeec Compare May 17, 2019 09:43
add empty line in the end of task

updated with console.log

add requirejs

add require

changed main.js

fix eslintrc

fix template
@GeFeSTTS GeFeSTTS requested a review from Kindest13 May 17, 2019 14:04
OstapDutkevych
OstapDutkevych previously approved these changes May 18, 2019
skiffua
skiffua previously approved these changes May 18, 2019
Kindest13
Kindest13 previously approved these changes May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants