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

JBIDE-18859 Basic Batch XML Editor (Tree form + source editor tabs) #277

Closed
wants to merge 1 commit into from

Conversation

@scabanovich
Copy link
Contributor

scabanovich commented Dec 2, 2014

Initial commit with complete 'batch' submodule structure and a test.

@scabanovich scabanovich force-pushed the scabanovich:jbide-18859 branch from 5b2f5a0 to 51d5c59 Dec 4, 2014
@@ -0,0 +1,245 @@
<?xml version="1.0" encoding="utf-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">

This comment has been minimized.

Copy link
@maxandersen

maxandersen Dec 9, 2014

Member

was this commited by accident ? seems unnecessary to add this before we got docs?

This comment has been minimized.

Copy link
@scabanovich

scabanovich Dec 9, 2014

Author Contributor

I was told that the first commit for batch should contain the entire structure required for build. This pom seems to me looking generic, not specific for a plugin that I used as a template. But if you think it should not be here before the docs are created, I will remove it.

@maxandersen
Copy link
Member

maxandersen commented Dec 9, 2014

this looks definitely better - but can we avoid committing arquillian specific image files ?

Initial commit with complete 'batch' submodule structure and a test.
@scabanovich scabanovich force-pushed the scabanovich:jbide-18859 branch from 51d5c59 to c321d79 Dec 11, 2014
@alexeykazakov
Copy link
Contributor

alexeykazakov commented Dec 12, 2014

Replaced by #279

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.