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

Some minor updates to pacman #315

Merged
merged 9 commits into from Apr 9, 2013
Merged

Some minor updates to pacman #315

merged 9 commits into from Apr 9, 2013

Conversation

Spuds
Copy link
Contributor

@Spuds Spuds commented Apr 7, 2013

No description provided.

Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>
Signed-off-by: Spuds <spuds@spudsdesign.com>

/**
* Checks if a package is installed or not
* If installed reutrnes an array of themes, db changes and versions associatd with
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: reutrnes

Signed-off-by: Spuds <spuds@spudsdesign.com>
! Fix spelling errors
! Make better use of the moved querys to reduce redundant redundancy
! wrong var used in the query

Signed-off-by: Spuds <spuds@spudsdesign.com>
@Spuds
Copy link
Contributor Author

Spuds commented Apr 8, 2013

I must say looking at that code and all the errors, I can only assume that Ema somehow has hijacked my ID ... or it was that cheap Sangiovese, I'm still a little blurry in one eye 😃

norv added a commit that referenced this pull request Apr 9, 2013
Some minor updates to pacman
@norv norv merged commit 5a27fb9 into elkarte:master Apr 9, 2013
@Spuds Spuds deleted the packageFactors branch April 9, 2013 20:45
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.

None yet

4 participants