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.

Sign up for GitHub

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

Jump to bottom

Would it be possible to report compilation and other pipeline error in a web page ? #26

Closed
cescoffier opened this issue Nov 24, 2013 · 6 comments
Closed

Would it be possible to report compilation and other pipeline error in a web page ? #26

cescoffier opened this issue Nov 24, 2013 · 6 comments
Labels
feature
Milestone
0.6

Comments

@cescoffier
Copy link
Member

No description provided.

@cescoffier cescoffier added this to the 0.5 milestone Apr 5, 2014
@cescoffier cescoffier modified the milestones: 0.6, 0.5 Apr 30, 2014
@cescoffier
Copy link
Member Author

I've started working on it, but it's more complex than expected. Retrieving and parsing error from Node modules is a bit tricky. So postponed the issue to the next release.

@romainPellerin
Copy link
Contributor

+1 it will be very useful in order to monitor your app build in a continuous delivery manner.

@cescoffier
Copy link
Member Author

@romainPellerin it's not really the purpose of this issue. Here it's just to display en error page when you get an issue in the pipeline (compilation issue or something related to the resource/source processing).

What you are mentioning is already supported as any error happening in the Maven build fails the build.

cescoffier added a commit that referenced this issue Jun 1, 2014
@cescoffier
Add information in the WatchingException to store the file, line, cha…
85c2e2e 
…racter and reason of the processing issue (#26).

Also add the support of the Java Compiler.

Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 1, 2014
@cescoffier
Enhanced the pipeline to propagate the watching exception to the wisd…
8819a0a 
…om server (#26)

Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 1, 2014
@cescoffier
Add a template presenting pipeline error and enhance the default erro…
b4d6d07 
…r handler to read the error.json and render the provided template (#26)

Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 2, 2014
@cescoffier
Improve the template and the pipeline error handling (#26)
b53ca3c 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 4, 2014
@cescoffier
Update pipeline error template (#26)
ca5ab24 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 4, 2014
@cescoffier
#26 - manage the case where the file is not specified
469101b 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 4, 2014
@cescoffier
#26 - Cleanup the pipeline error directory when the pipeline starts.
2798745 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 4, 2014
@cescoffier
#26 - add compilation error support for Less
85efb1a 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 4, 2014
@cescoffier
Distinguish message and title in the error report (#26)
d440d32 
Signed-off-by: Clement Escoffier <[email protected]>
@romainPellerin
Copy link
Contributor

@cescoffier that's exactly what I mean, it will be very useful to get these errors trace from the app at runtime and not at build time, in order to build dynamic supervision features.

I am seeing you are already on the way to implement it, will be awesome!

cescoffier added a commit that referenced this issue Jun 5, 2014
@cescoffier
Fix the last commit (#26). It was my test...
5734387 
Signed-off-by: Clement Escoffier <[email protected]>
cescoffier added a commit that referenced this issue Jun 5, 2014
@cescoffier
Add test to the default error handler on the pipeline error proper ha…
7757e89 
…ndling (#26)

Signed-off-by: Clement Escoffier <[email protected]>
@cescoffier
Copy link
Member Author

Done !

@romainPellerin
Copy link
Contributor

Terrific !!!! Going to introduce it asap in my use case!

Romain Pellerin | Fondateur - Gérant

1 rue Fleming - 17000 La Rochelle | www.ubidreams.com

+33 (0)5 16 49 04 84 | 06 89 28 95 80

*Fondateur & Mentor du projet libre NANOKO *| www.nanoko.org

Blog http://romainpellerin.fr/ | Linkedin
http://www.linkedin.com/in/romainpellerin | Google+
http://plus.google.com/117099266649367147445/posts | Twitter
http://twitter.com/#!/rom1_ubidreams | ID Skype: rom1_ubidreams

On Fri, Jun 6, 2014 at 10:56 AM, Clement Escoffier <[email protected]

wrote:

Done !

—
Reply to this email directly or view it on GitHub
#26 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
0.6
Development

No branches or pull requests
2 participants
@cescoffier @romainPellerin