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

Added delete operation in sdb.sqlite3 module. #51479

Merged
merged 2 commits into from Feb 6, 2019
Merged

Conversation

jwezel
Copy link

@jwezel jwezel commented Feb 2, 2019

Amends sdb.sqlite3 module with delete operation.

Tests written?

No

Commits signed with GPG?

No

jwezel added 2 commits Feb 2, 2019
- Comment string
- Source formatting
@garethgreenaway
Copy link
Member

@garethgreenaway garethgreenaway commented Feb 6, 2019

@jwezel Thanks for the PR, can you please add some tests for this new functionality?

dwoz
dwoz approved these changes Feb 6, 2019
@dwoz dwoz merged commit 0e00133 into saltstack:develop Feb 6, 2019
10 checks passed
@waynew waynew added this to PR needs port to master in PRs to port to master Oct 24, 2019
alexey-zhukovin pushed a commit to alexey-zhukovin/salt that referenced this issue Apr 29, 2020
@alexey-zhukovin alexey-zhukovin added the has master-port label Apr 29, 2020
@alexey-zhukovin alexey-zhukovin moved this from PR needs port to master to PR has port to master in PRs to port to master Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has master-port
Projects
PRs to port to master
  
PR has port to master
Development

Successfully merging this pull request may close these issues.

None yet

4 participants