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

XmlRepomd.java:43-51: Refactor this class so it uses... #105

Closed
0pdd opened this issue Apr 13, 2020 · 14 comments
Closed

XmlRepomd.java:43-51: Refactor this class so it uses... #105

0pdd opened this issue Apr 13, 2020 · 14 comments

Comments

@0pdd
Copy link

0pdd commented Apr 13, 2020

The puzzle 81-6d01bf6b from #81 has to be resolved:

* @todo #81:30min Refactor this class so it uses XmlFile instead of using
* a XMLStreamWriter and a Path. It is not straightforward because it exposes
* the Path via the path() which is now hidden in the XmlFile object.
* We must keep the path hidden there while at the same time allow for
* Repository to be responsible of triggering the save of the repomd file
* after all the metadata files (declared in Rpm) have wrote themselves to
* repomd as expected. One solution would be to delegate to the xml files
* and thus to XmlFile the act of moving itself in the desired place: in
* that case the way other metadata files are saved should be adapted.

The puzzle was created by Victor Noël on 11-Apr-20.

Estimate: 30 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0crat
Copy link

0crat commented Apr 13, 2020

@g4s8/z please, pay attention to this issue

@0crat 0crat removed their assignment Apr 13, 2020
@0crat
Copy link

0crat commented Apr 14, 2020

The job #105 assigned to @victornoel/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@victornoel
Copy link
Contributor

@g4s8 this is not easy at all... do you know if you will tackle #99 soon?

I think waiting for the choices you will make there is going to be safer before tackling this one...

@g4s8
Copy link
Member

g4s8 commented Apr 15, 2020

@victornoel ok, let's wait for #99

@victornoel
Copy link
Contributor

@0crat wait for #99 to be done first

@0crat 0crat added the waiting label Apr 15, 2020
@victornoel
Copy link
Contributor

@g4s8 is this issue and #99 still in scope nowadays?

@victornoel
Copy link
Contributor

@g4s8 ping :)

@victornoel
Copy link
Contributor

@0crat refuse

@0crat
Copy link

0crat commented May 13, 2020

@0crat refuse (here)

@victornoel The user @victornoel/z resigned from #105, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link

0crat commented May 13, 2020

The job #105 assigned to @paulodamaso/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@paulodamaso
Copy link
Contributor

@g4s8 Let's wait for #99, you can assign it to me if you want

@paulodamaso
Copy link
Contributor

@0crat wait for #99

@0pdd
Copy link
Author

0pdd commented Jun 17, 2020

The puzzle 81-6d01bf6b has disappeared from the source code, that's why I closed this issue.

@0crat
Copy link

0crat commented Jun 17, 2020

Job gh:artipie/rpm-adapter#105 is not assigned, can't get performer

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

No branches or pull requests

5 participants