This revision was automatically updated to reflect the committed changes.
Closed by commit R175:97139b80e80b: Nicaragua Holidays (authored by carguello,
committed by cgiboudeaux).
CHANGED PRIOR TO COMMIT
https://phabricator.kde.org/D28891?vs=80322&id=80452#toc
REPOSITORY
R175 KHolidays
CHAN
winterz accepted this revision.
winterz added a comment.
This revision is now accepted and ready to land.
other than the indentation in the .qrc file this is good to go.. tests pass
would appreciate if someone would commit this for us. (after fixing the
indentation)
REPOSITORY
R175 KH
winterz added inline comments.
INLINE COMMENTS
> holidays.qrc:101
> plan2/holiday_mx_es
> -plan2/holiday_na_en-gb
> + plan2/holiday_na_en-gb
> plan2/holiday_nc_fr
realign the indentation
> holidays.qrc:103
> plan2/holiday_nc_fr
> + plan2/holida
ngraham added a comment.
You're very welcome. Thank you for the patch!
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
carguello added a comment.
Thank you for your time and support. 🙂🙂
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
ngraham added a comment.
Great job! I'll hand it over to @winterz and other reviewers now.
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
carguello added a comment.
Okay, I'm confident it went through this time
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
carguello updated this revision to Diff 80322.
carguello added a comment.
Adding Nicaragua Holidays file back
REPOSITORY
R175 KHolidays
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28891?vs=80319&id=80322
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D28891
AF
ngraham added a comment.
git add [the path to the new file you want to add]`
arc diff --update D28891
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michael
carguello added a comment.
😶
How can I add it back? Sorry this is my first time using git and Phabricator
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, mich
ngraham added a comment.
Now the new file you wanted to add is gone. :)
With Phabricator, you need to include all commits in your patch, not just the
latest one.
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngr
carguello updated this revision to Diff 80319.
carguello added a comment.
Previously messed up holidays/holidays.qrc I have fixed it now
REPOSITORY
R175 KHolidays
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28891?vs=80318&id=80319
BRANCH
master
REVISION DETAIL
https://pha
carguello added a comment.
I think I messed up 😔
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
carguello updated this revision to Diff 80318.
carguello added a comment.
Remove file extension .qrc per instructions
REPOSITORY
R175 KHolidays
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28891?vs=80309&id=80318
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D2
ngraham removed a reviewer: VDG.
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, winterz, #frameworks
Cc: ngraham, kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, bruns
ngraham added reviewers: winterz, Frameworks.
ngraham added a comment.
Thanks for the patch! You need to update the `holidays/holidays.qrc` file
too, to mention this new file that you're adding. Also I don't think you need
to have the `.qrc` extension on the file you're adding.
REPOSITORY
carguello added a reviewer: VDG.
REPOSITORY
R175 KHolidays
REVISION DETAIL
https://phabricator.kde.org/D28891
To: carguello, #vdg
Cc: kde-frameworks-devel, #vdg, LeGast00n, cblack, michaelh, ngraham, bruns
carguello created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
carguello requested review of this revision.
REVISION SUMMARY
Added file containing Nicaragua holidays in holidays/plan2/holiday_ni_es
REPOSITORY
R175 KHolidays
BRANCH
mast
18 matches
Mail list logo