better logging; run_once check for pull state
This commit is contained in:
parent
f5b84b530c
commit
ba8119903e
@ -27,6 +27,7 @@ class Routine extends Injectable {
|
||||
|
||||
let step_no = 1
|
||||
for ( const step of result.steps ) {
|
||||
if ( with_logging ) this.output.info('', 0)
|
||||
this.output.info(`(${step_no}/${result.steps.length}) ${step.step.display()}`, with_logging ? 0 : 10)
|
||||
step_no += 1
|
||||
|
||||
|
@ -13,10 +13,13 @@ class PermissionState extends State {
|
||||
}
|
||||
|
||||
async check() {
|
||||
const path = await this._path()
|
||||
const permissions = `${await path.permissions()}`.trim()
|
||||
const target = `${this._config.level}`.trim()
|
||||
return permissions === target
|
||||
try {
|
||||
const path = await this._path()
|
||||
const permissions = `${await path.permissions()}`.trim()
|
||||
const target = `${this._config.level}`.trim()
|
||||
return permissions === target
|
||||
} catch (e) {}
|
||||
return false
|
||||
}
|
||||
|
||||
async reverse() {
|
||||
|
@ -1,6 +1,8 @@
|
||||
const AbstractGitState = require('./AbstractGitState')
|
||||
|
||||
class PullState extends AbstractGitState {
|
||||
#ran_once = false
|
||||
|
||||
static get services() {
|
||||
return [...super.services, 'output']
|
||||
}
|
||||
@ -9,11 +11,12 @@ class PullState extends AbstractGitState {
|
||||
if ( !(await this.check()) ) {
|
||||
const repo = await this._repo()
|
||||
await repo.pull(this._config.target)
|
||||
this.#ran_once = true
|
||||
}
|
||||
}
|
||||
|
||||
async check() {
|
||||
return true // TODO support a better check here
|
||||
return this.#ran_once // TODO support a better check here
|
||||
}
|
||||
|
||||
async reverse() {
|
||||
|
Loading…
Reference in New Issue
Block a user