Skip to content

Conversation

@revolj
Copy link
Collaborator

@revolj revolj commented Jan 14, 2026

No description provided.

@revolj revolj requested a review from parmentf January 14, 2026 13:06
@revolj revolj self-assigned this Jan 14, 2026
@revolj revolj requested a review from Luc-Ank January 15, 2026 14:41
Comment on lines +27 to +30

# if pid == "" and data["value"] != []:
# pid = data[[x for x in list(data.keys()) if x.startswith("PID")][0]][5:]

Copy link
Contributor

Choose a reason for hiding this comment

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

polish: ‏Dead code

We should remove unused code

Suggested change
# if pid == "" and data["value"] != []:
# pid = data[[x for x in list(data.keys()) if x.startswith("PID")][0]][5:]

post.summary = Récupération du résultat sous format xml/gexf et png, dans un tar.gz. Pour utiliser le fichier gexf, assurez-vous de changer l'extension en `.gexf`.
post.description = Récupération du résultat sous format xml/gexf et png, dans un tar.gz. Pour utiliser le fichier gexf, assurez-vous de changer l'extension en `.gexf`.
post.tags.0 = data-graph
post.tags.0 = retrieves
Copy link
Contributor

Choose a reason for hiding this comment

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

👏 praise: ‏Good idea

Comment on lines 3 to 4
# Use it to test:
# npx hurl --test data-computer/tests.hurl
Copy link
Contributor

Choose a reason for hiding this comment

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

polish: ‏Remove duplicate part

Suggested change
# Use it to test:
# npx hurl --test data-computer/tests.hurl

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants