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

Refactor fetchText to use host.fetchText function #886

Merged
merged 4 commits into from
Nov 21, 2024
Merged

Conversation

pelikhan
Copy link
Member

@pelikhan pelikhan commented Nov 21, 2024

Migrate the usage of fetchText to host.fetchText across documentation and codebase for consistency and improved functionality. fix for #885


  • 🔄 Refactoring Fetch Functions: Updated the fetchText function to host.fetchText across multiple documentation and script files to standardize the API usage and improve proxy and retry capabilities.
  • 🛠️ Enhancements to Fetch Utilities: Introduced host.fetch as a wrapper around the global fetch function with additional proxy support and retry mechanisms for more robust HTTP requests.
  • 🧩 Improved GitHub Integration: Enhanced the githubParseEnv function to optionally resolve issues automatically, providing better integration with GitHub environments.
  • 🐛 Debugging Improvements: Added console logging for GitHub connection information to aid in debugging and traceability.
  • 📝 Documentation Updates: Revised documentation to reflect changes in function names and usage patterns, ensuring consistency and clarity for developers.
  • ⚙️ Type Definitions: Updated type definitions for fetch options, including retry and delay configurations, to support the new fetch functionalities.
  • 📜 Sample Script Adjustments: Modified sample scripts to utilize the updated host.fetch and host.fetchText methods, demonstrating their usage and benefits.

generated by prd

@pelikhan
Copy link
Member Author

@sahuguet host.fetch calls createFetch under the hood

Copy link

Investigator report

, 2859d8f

generated by gai

@pelikhan pelikhan merged commit 30d6a0c into main Nov 21, 2024
10 checks passed
@pelikhan pelikhan deleted the host.fetch branch November 21, 2024 18:58
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.

1 participant