Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0043000
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] A. Platformmajorhave not tried2020-01-24 12:372020-02-14 09:32
ReporterjarmendarizView Statuspublic 
Assigned Tocberner 
PriorityhighResolutionfixedFixed in Version3.0PR20Q2
StatusclosedFix in branchFixed in SCM revision6140b72f414d
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned Toalostale
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0043000: Migrate current Mercurial hooks to Git hooks

DescriptionCurrently we provide Mercurial hooks to automatically check for javascript linting/formatting errors before creating a commit.
Git also support hooks in various stages [1], but with the different there can only be one script for each stage. It'd be convenient to have a mechanism to execute multiple scripts for the same hook.

[1] https://git-scm.com/docs/githooks [^]
Steps To Reproduce* Modify a javascript file. For instance: modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
* Add a line that will cause a linting error:
var a = 1;
* Stage and create a commit

Before the commit is created, ESLint should run an notify there's an error.

TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to feature request 0041898 acknowledgedTriage Platform Base support git 

-  Notes
(0117397)
alostale (viewer)
2020-01-31 09:29

 MR: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/37 [^]
(0117556)
hgbot (developer)
2020-02-07 11:54

 Repository: erp/devel/pi
Changeset: 6140b72f414d17a85ffc31d292d71705fa4a01f9
Author: Cristian Berner <cristian.berner <at> openbravo.com>
Date: Mon Jan 27 17:04:20 2020 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/6140b72f414d17a85ffc31d292d71705fa4a01f9 [^]

Fixes ISSUE-43000: Adapted Mercurial hooks to be used in git

Hooks are also moved to a .githooks folder to have them grouped and ready
to be set as the project hooks folder running:

'git config core.hooksPath .gitHooks'

As there can be only one pre-commit script, a pre-commit.d folder were
created to have both scripts as an ordered list which will be executed in
order by the pre-commit script.

---
M modules/org.openbravo.client.kernel/jsformatter/README.md
M modules/org.openbravo.client.kernel/jslint/README.md
A .githooks/pre-commit
A .githooks/pre-commit.d/00-jslint
A .githooks/pre-commit.d/01-jsformatter
---

- Issue History
Date Modified Username Field Change
2020-01-24 12:37 jarmendariz New Issue
2020-01-24 12:37 jarmendariz Assigned To => platform
2020-01-24 12:37 jarmendariz OBNetwork customer => No
2020-01-24 12:37 jarmendariz Modules => Core
2020-01-24 12:37 jarmendariz Triggers an Emergency Pack => No
2020-01-24 12:45 jarmendariz Relationship added related to 0041898
2020-01-31 09:29 alostale Note Added: 0117397
2020-01-31 09:29 alostale Assigned To platform => cberner
2020-02-07 11:54 hgbot Checkin
2020-02-07 11:54 hgbot Note Added: 0117556
2020-02-07 11:54 hgbot Status new => resolved
2020-02-07 11:54 hgbot Resolution open => fixed
2020-02-07 11:54 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/6140b72f414d17a85ffc31d292d71705fa4a01f9 [^]
2020-02-07 12:44 cberner Review Assigned To => alostale
2020-02-14 09:32 alostale Status resolved => closed
2020-02-14 09:32 alostale Fixed in Version => 3.0PR20Q2

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker