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

svn status can now include changelists' items #133

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

ALERTua
Copy link

@ALERTua ALERTua commented Mar 5, 2019

Idea products often create SVN changelists while performing basic file operations. Your SVN status only parses the files not included in any changelist. This is 'svn status --xml' output for such case:

<?xml version='1.0' encoding='utf8'?>
<status>
	<target path="E:\Source\folder">
		<entry path="E:\Source\folder\scripts">
			<wc-status item="unversioned" props="none">
			</wc-status>
		</entry>
		<entry path="E:\Source\folder\other_scripts\test6.xml">
			<wc-status item="unversioned" props="none">
			</wc-status>
		</entry>
	</target>
	<changelist name="my_changelist">
		<entry path="E:\Source\folder\miscs\my.py">
			<wc-status item="modified" props="none" revision="110">
				<commit revision="108">
					<author>someone</author>
					<date>2019-02-11T19:16:48.900000Z</date>
				</commit>
			</wc-status>
		</entry>
		<entry path="E:\Source\folder\miscs\acc.py">
			<wc-status item="modified" props="none" revision="110">
				<commit revision="107">
					<author>not_me</author>
					<date>2018-12-18T17:07:58.188000Z</date>
				</commit>
			</wc-status>
		</entry>
	</changelist>
</status>

I added an option to include changelist/entry xml blocks in parsing svn status.

@coveralls
Copy link

Coverage Status

Coverage decreased (-28.7%) to 71.268% when pulling 761162e on ALERTua:master into 0c222a9 on dsoprea:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-28.7%) to 71.268% when pulling 761162e on ALERTua:master into 0c222a9 on dsoprea:master.

@coveralls
Copy link

coveralls commented Mar 5, 2019

Coverage Status

Coverage decreased (-28.7%) to 71.268% when pulling 0475d9c on ALERTua:master into 0c222a9 on dsoprea:master.

@dsoprea
Copy link
Owner

dsoprea commented May 30, 2019

Please fix the coverage drop and drop the version bump, squash your changes and make sure those exclamation points dont stay in the commit messages (and capitalized the initial while you're at it), and we can review.

Sorry about the delay. I thought I'd already asked about the coverage drop.

@dsoprea dsoprea force-pushed the master branch 2 times, most recently from 544f7da to 230a2cd Compare February 1, 2020 22:10
@idbrii
Copy link

idbrii commented Nov 24, 2020

#162 is an alternative implementation: instead of an argument to add changelists, it always gets changelists and adds the changelist to the returned status object. This makes it possible to see which files are in which changelists.

Aleksei Rubashev and others added 2 commits February 23, 2022 11:46
# Conflicts:
#	svn/__init__.py
#	svn/common.py
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.

4 participants