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

Merge conflict resolution related to DES-2131 #105

Conversation

nathanfranklin
Copy link
Collaborator

@nathanfranklin nathanfranklin commented Sep 8, 2022

Overview:

Resolve the merge conflicts.

Notes

Was looking at the react testing and wanted the github actions workflow in #95 so resolved conflicts locally while working on that. See #95 as I realized that the linting/lint-fixing isn't aligned with the changes in that PR.

* Disable or remove project delete buttons

If user can't delete, then hide/disable the button

* Improve warning message
Copy link
Contributor

@duckonomy duckonomy left a comment

Choose a reason for hiding this comment

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

LGTM!

@duckonomy duckonomy merged commit ee76906 into task/DES-2131-add-github-actions-ci Sep 27, 2022
duckonomy added a commit that referenced this pull request Mar 2, 2023
* Fix linting and testing and add github ci.

* Update README for linting instructions.

* Fix jwt.js.

* Fix test config.

* Add custom launcher for karma.

* Add custom launcher for karma.

* Update for test

* Add new start npm script instructions for to README.

* Adjust workflow to use angular folder (#102)

* Merge conflict resolution related to DES-2131 (#105)

* Task/DES-1996 restrict deletion of projects to admins/creators (#101)

* Disable or remove project delete buttons

If user can't delete, then hide/disable the button

* Improve warning message

* Fix linting

* Remove comment

* Replace tslint prettier with prettier.

* Run prettier.

* Run ng lint.

* Run prettier with new max length.

* Run eslint.

* Add prettierc.

* Add prettier npm script.

* Fix max-line-length parameter.

* Add fix options to linter and prettier.

* Add prettier as dev dependency.

* Fix prettier check.

* Run prettier.

---------

Co-authored-by: Nathan Franklin <[email protected]>
@nathanfranklin nathanfranklin deleted the task/DES-2131-add-github-actions-ci---merge-conflict branch April 10, 2023 20:58
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.

2 participants