Change libkonq/PLUGINS and libkonq/SERVICEMENUS to Markdown format #292
Merged
SlavekB
merged 1 commits from issue/290/servicemenus-ntfix
into master
2 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'issue/290/servicemenus-ntfix'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Signed-off-by: bobsmith bobsmith432@tutanota.com
That's not good. It is desirable that the file is really use Markdown formatting – see the comment below.
At the same time, there is no good message in git commit. There should be stated what is the subject of the commit, but there is no point in providing a call for testing. This belongs to the Pull-Request comment.
A suitable git commit messages could be for example:
konqueror and kdesktop), without using the file associations.
This file has been converted to Markdown, in order to ensure compatibility with the Windows NT kernel
- bobsmith
It is unnecessary to insert such a comment into this file.
It should be more important to edit the file to use Markdown formatting. You can see that now it really does not look good – especially the part with the desktop file:
77fb786043/libkonq/SERVICEMENUS.md
So I'm using the Abricotine editor, and it looks just fine in that for some reason. I do agree I probably shouldn't have slipped in the comment.
You can adjust the commit with
git commit --amend --no-edit
and then usegit push -f origin HEAD
to update the code here.@bobsmith
thanks for your first PR!! Slavek and I will share guidelines on what is good and not-so-good practice for commits, hope it will be useful.
77fb786043
to2c9187584a
2 years agoIn order to move forward, I have made an editing of the
SERVICEMENUS
file to make use of Markdown formatting. At the same time, I also made aPLUGINS
file formatting. See:2c9187584a/libkonq/SERVICEMENUS.md
and2c9187584a/libkonq/PLUGINS.md
WIP: Should fix #290, but other Windows users please testto WIP: Change libkonq/PLUGINS and libkonq/SERVICEMENUS to Markdown format 2 years agoLooks good
WIP: Change libkonq/PLUGINS and libkonq/SERVICEMENUS to Markdown formatto Change libkonq/PLUGINS and libkonq/SERVICEMENUS to Markdown format 2 years agoFormatting adjustments have been resolved, so nothing prevents merge.
2c9187584a
into master 2 years agoThank you for this!
Reviewers
2c9187584a
.