vikinghug / VikingUI

A fork of VikingUI
3 stars 5 forks source link

Update VikingInventoryBag.lua #25

Closed Zeirk closed 8 years ago

mkaito commented 8 years ago

Woah. chuckle

You can put all that into a single PR, man. PRs are logical units, and in no way tied to files.

Are you using the github web interface for this? I find it much easier to use the git commandline, to be honest. Create a feature branch, make your changes in one or multiple commits, as you see fit to organize changes (doesn't need to be per-file either, I like self-contained logical change sets), and then send a pull request for the entire branch.

I assume you can do the same via web interface, although I've never done that, and so I can not provide instructions.

I'm going to close these PRs. I'm a bit of a stickler for clean commit history.