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

WIP spires: support for particles #39

Closed
wants to merge 1 commit into from
Closed

WIP spires: support for particles #39

wants to merge 1 commit into from

Conversation

Panos512
Copy link
Contributor

@Panos512 Panos512 commented Oct 6, 2015

Signed-off-by: Panos Paparrigopoulos [email protected]

* Adds a list of particles.

* Identifies particles as "particle" instead of "value".
  (closes #38)

Signed-off-by: Panos Paparrigopoulos <[email protected]>
@jirikuncar jirikuncar added this to the v0.4.0 milestone Oct 6, 2015
@@ -0,0 +1,783 @@
# -*- coding: utf-8 -*-
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't spires contrib config enough?

@jirikuncar
Copy link
Member

@Panos512 all changes should touch only invenio_query_parser/contrib/spires/ folder.

@jirikuncar jirikuncar changed the title WIP parser: Particles identification WIP spires: support for particles Nov 9, 2015
@Panos512
Copy link
Contributor Author

Panos512 commented Nov 9, 2015

@jirikuncar Thanks for the suggestions! I will get back to it as soon as possible.

@jirikuncar jirikuncar modified the milestones: v0.4.0, v0.5.0 Nov 10, 2015
@egabancho egabancho modified the milestones: someday, v0.5.0 Feb 29, 2016
@jmartinm
Copy link
Member

jmartinm commented Oct 4, 2016

ping @Panos512 what was the status of this PR?

@Panos512
Copy link
Contributor Author

Panos512 commented Oct 6, 2016

@jmartinm I can pick it up again if we still need it.

@egabancho
Copy link
Member

@kaplun you mentioned Panos left already and this PR has been sleeping for a year or so, do you think you'll have time to take look at it should we close it for now?

@jacquerie jacquerie self-assigned this Nov 21, 2016
@jacquerie
Copy link

jacquerie commented Nov 21, 2016

No, I'm going to take care of it (but not now, because I've got a lot on my plate currently...).

@lnielsen
Copy link
Member

Closing PR as it's been stale for a long time.

@lnielsen lnielsen closed this Feb 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

search: support for particles
6 participants