Typo fixes for umbrello
ClosedPublic

Authored by okellogg on Jan 4 2020, 1:28 PM.

Details

Reviewers
habacker
Summary

Diff was created against umbrello/master @ 51ed1e9.

Test Plan

Complete rebuild was done.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
okellogg created this revision.Jan 4 2020, 1:28 PM
Restricted Application added a subscriber: umbrello-devel. ยท View Herald TranscriptJan 4 2020, 1:28 PM
okellogg requested review of this revision.Jan 4 2020, 1:28 PM
habacker accepted this revision.Jan 5 2020, 11:23 AM
This revision is now accepted and ready to land.Jan 5 2020, 11:23 AM

BTW: there is a new way to submit code changes to umbrello by creating gitlab merge requests at https://invent.kde.org/kde/umbrello/merge_requests

Please commit this patch yourself, otherwise I will be listed as author, as the unpatched arc does not use the original author or create a merge request.

If you do not know how to create a merge request, you must do so:

  1. register at https://invent.kde.org.
  2. fork https://invent.kde.org/kde/umbrello into your home project (click on button 'fork')
  3. git clone your forked umbrello project
  4. apply changes to the desired branch of the checked-out git repository
  5. push the current branch into your forked umbrello project

this will display a link to open a corresponding merge request.

I followed the steps you described and created a merge request:
https://invent.kde.org/kde/umbrello/merge_requests/1

The Merge button appears greyed out for me saying

Ready to be merged automatically. Ask someone with write access to this repository to merge this request

Thanks,

Oliver

habacker closed this revision.Jan 5 2020, 8:03 PM

I followed the steps you described and created a merge request:
https://invent.kde.org/kde/umbrello/merge_requests/1

I merged this request into git master branch of kde/umbrello, thank you for your contribution.

The Merge button appears greyed out for me saying

Ready to be merged automatically. Ask someone with write access to this repository to merge this request

This is because you are not a member of the kde/umbrello project on invent.kde.org. File a member adding request at https://community.kde.org/Sysadmin for getting write access.