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

test pull request #137

Open
wants to merge 74 commits into
base: master
Choose a base branch
from
Open

test pull request #137

wants to merge 74 commits into from

Conversation

ChadLiberty
Copy link

this is a test

sDriskell and others added 30 commits March 31, 2020 07:26
Added workflow functionality to project
Using different workflow action to test for project.
Remove previous workflow action to test different workflow
My initial commit.  Fixed font size issue.
Revert "My initial commit.  Fixed font size issue."
cleaned up commented out code, seemed unused
I had some changes in Gitkraken that had not been committed.  Trying this for first time.
@oroulet
Copy link
Contributor

oroulet commented Apr 20, 2020

You are welcome to make a real one. Looks like mainly formatting but that is fine too

@sDriskell
Copy link

sDriskell commented May 14, 2020

You are welcome to make a real one. Looks like mainly formatting but that is fine too

If you're interested, we fixed the issue with the due date being invalid format due to the empty space. I made a change in CompletionPopup.qml and removed the empty space that was being placed after "due:" so the htmlizer.py would work properly with parsing the key words.

It's in my fork of the project.

Also, please excuse the plethora of commits and garbage commit comments; this was a college course assignment and there was a lot of trying to figure out how to use gitKraken, creating branches, merges, etc. So sorry that it's a litter of crap but you shouldn't see anymore for the time being.

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.

5 participants