Add get_processes() to wrap v1/system/processes

Fermée
#531 0 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@jmwoliver y travaille déjà.

Depuis le 6/11/2026.

  • #533 par @jmwoliver — fusionnée

Évaluation

Cette issue n'a pas encore été évaluée.

Description

I think we want a new function name than get_procs(), the old one. The new function should call GET v1/system/processes with Accept: text/csv to get a CSV download and then reading the data in should be straightforward. If that 404s, then we're pointing at old Connect, so call the old API (justget_procs()), and map the old column names to the new ones. We have a few of these "call the new API and if it fails call the old one" setups in the package already.

The new API supports a few filtering params that we could add as arguments to get_processes()

get_procs() should call .Deprecated() (or if there is a fancier lifecycle:: function we can use that since we already depend on it) and tell people to use get_processes() instead. (Though we wouldn't want to raise the warning if we're calling the logic from within get_processes() as the fallback, so either suppress the warning there or factor out the logic that does the requests and formatting.)

Langage dominant
R
Étoiles
54
Forks
27
Merge moyen
1 j 28 min
PR mergées (30 j)
3

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de posit-dev/connectapi

Toutes les issues de posit-dev/connectapi

Issues similaires

Plus d'issues R

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.