Project

General

Profile

Actions

Architecture #5311

closed

Move code related to promise generation from cf-clerck to Rudder

Added by François ARMAND over 10 years ago. Updated over 9 years ago.

Status:
Rejected
Priority:
1 (highest)
Category:
Architecture - Code maintenance
Target version:
Effort required:
Name check:
Fix check:
Regression:

Description

cf-clerk is today responsible of 2 main things:

- reading technique from repos (FS, Git)
- writing promises to the FS from context.

But Rudder is also partially responsible for that second task, as it is the one who knows what should be written, where, with what values.

The current separation of code between Rudder and cf-clerk makes no sense at all.

So for now, move all the promise generation logic into Rudder, so that we could rebuild a coherent whole from that point, and perhaps split-it again in the future from Rudder.

The gain is:
- simplification of actual code ;
- less code (quicker to build) ;
- modification in only one place, so 2 times less pull requests for code related to that.


Related issues 1 (0 open1 closed)

Is duplicate of Rudder - Architecture #6948: Merge cf-clerk into RudderReleasedVincent MEMBRÉ2015-07-15Actions
Actions #1

Updated by Vincent MEMBRÉ over 10 years ago

  • Description updated (diff)

Already love that <3

Actions #2

Updated by Matthieu CERDA about 10 years ago

  • Target version changed from 140 to 3.0.0~beta1
Actions #3

Updated by Jonathan CLARKE about 10 years ago

  • Target version changed from 3.0.0~beta1 to 3.1.0~beta1
Actions #4

Updated by Benoît PECCATTE over 9 years ago

  • Status changed from 8 to New
Actions #5

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 3.1.0~beta1 to 3.1.0~rc1
Actions #6

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 3.1.0~rc1 to 3.1.0
Actions #7

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 3.1.0 to 3.1.1
Actions #8

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 3.1.1 to 3.1.2
Actions #9

Updated by Jonathan CLARKE over 9 years ago

  • Status changed from New to Rejected

This is a duplicate of #6948. cf-clerk has been merged into Rudder on master branch, and will be released as such in Rudder 3.2 (may be named Rudder 4.0)

Actions #10

Updated by Jonathan CLARKE over 9 years ago

Actions

Also available in: Atom PDF