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

Remove get_item and set_item from pyrflx packages #783

Closed
jklmnn opened this issue Sep 28, 2021 · 0 comments · Fixed by #785
Closed

Remove get_item and set_item from pyrflx packages #783

jklmnn opened this issue Sep 28, 2021 · 0 comments · Fixed by #785
Assignees
@jklmnn
Labels
pyrflx Related to pyrflx package (Legacy Python API)
Projects
RecordFlux 0.6

Comments

@jklmnn
Copy link
Member

jklmnn commented Sep 28, 2021

Remove the get_item and set_item API from pyrflx packages. To support the iterator for the validation tool implement the iterator interface.
@jklmnn jklmnn created this issue from a note in RecordFlux 0.6 (To do) Sep 28, 2021
@jklmnn jklmnn self-assigned this Sep 28, 2021
@jklmnn jklmnn added the pyrflx Related to pyrflx package (Legacy Python API) label Sep 28, 2021
@jklmnn jklmnn mentioned this issue Sep 28, 2021
Merged
@jklmnn jklmnn moved this from To do to In progress in RecordFlux 0.6 Sep 28, 2021
jklmnn added a commit that referenced this issue Sep 28, 2021
@jklmnn
Remove __setitem__ from PyRFLX package API
eb851df 
ref #783
@jklmnn jklmnn mentioned this issue Sep 28, 2021
Closed
jklmnn added a commit that referenced this issue Sep 28, 2021
@jklmnn
Remove __setitem__ from PyRFLX package API
f45e7d9 
ref #783
jklmnn added a commit that referenced this issue Sep 28, 2021
@jklmnn
Remove __setitem__ from PyRFLX package API
dc702ac 
ref #783
jklmnn added a commit that referenced this issue Sep 28, 2021
@jklmnn
Remove __getitem__ from PyRFLX
6340b77 
ref #783
@jklmnn jklmnn mentioned this issue Sep 28, 2021
Merged
jklmnn added a commit that referenced this issue Sep 28, 2021
@jklmnn
Update specifications
0194a47 
ref #783
jklmnn added a commit that referenced this issue Sep 29, 2021
@jklmnn
Remove __getitem__ from PyRFLX
bd039dc 
ref #783
jklmnn added a commit that referenced this issue Sep 29, 2021
@jklmnn
Update specifications
1f3ed6f 
ref #783
@treiher treiher linked a pull request Sep 30, 2021 that will close this issue
Remove __getitem__ from PyRFLX #785
Merged
@treiher treiher closed this as completed Sep 30, 2021
RecordFlux 0.6 automation moved this from In progress to Merged Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jklmnn jklmnn

Labels
pyrflx Related to pyrflx package (Legacy Python API)
Projects
No open projects
RecordFlux 0.6
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove __getitem__ from PyRFLX AdaCore/RecordFlux
2 participants
@jklmnn @treiher