Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CodeQL config #69

Closed
wants to merge 3 commits into from
Closed

CodeQL config #69

wants to merge 3 commits into from

Conversation

akshatgarg12
Copy link
Contributor

What kind of change does this PR introduce?

Issue Number:

Fixes #62

Did you add tests for your changes?

Snapshots/Videos:

If relevant, did you update the documentation?

Summary

Does this PR introduce a breaking change?

Other information

similar to #67
Have you read the contributing guide?

Yes

palisadoes and others added 3 commits May 30, 2021 10:52
* making welcome page and login page

* Update Navbar.tsx

* json file

* del

* changed

* changed the fromatting

* added test

* formatting

* changes in position

* added test for login

* Fixes to UI

* Fixes to login UI

* changed the UI

* Delete package-lock.json

* changes required changed

* chnging

* added test

* Update LoginPage.test.tsx

* changed test

* fixing flow

* Update LoginPage.test.tsx

* Delete package-lock.json

* changes

* Update Navbar.test.tsx

Co-authored-by: DangaRanga <[email protected]>
Copy link
Contributor

@palisadoes palisadoes left a comment

Choose a reason for hiding this comment

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

@akshatgarg12 We just want changes in the .github/workflows directory. You are overwriting all of the master branch with new code.

@palisadoes
Copy link
Contributor

@akshatgarg12 Focus on the develop branch with the other issues. The GitHub action is running OK in develop. It'll get into the other branches eventually.

@palisadoes
Copy link
Contributor

@akshatgarg12 Join our slack organization It'll help a lot.

@akshatgarg12
Copy link
Contributor Author

@palisadoes Sure sir.

@akshatgarg12
Copy link
Contributor Author

@palisadoes I have already created a PR for the alpha branch PR : #71 , shall i create one for master aswell regarding this #62 ?

@palisadoes
Copy link
Contributor

@akshatgarg12 Yes, go ahead.

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.

3 participants