| Age | Commit message (Collapse) | Author |
|---|
|
Introduce `wttrin-mode`, a derived major mode for displaying weather
information from wttr.in. This includes keybindings for refreshing
and querying new locations, and sets up the buffer with a custom
font and read-only settings. Existing display logic has been
refactored to utilize this mode for improved usability and code
organization.
|
|
Introduce asynchronous data fetching to the wttrin.el Emacs package.
This enhancement avoids blocking Emacs during data retrieval by
using `url-retrieve` for async calls. The behavior is controlled via
a new customizable variable `wttrin-use-async`. Tests have been
added for the new async behavior to ensure proper functionality.feat:makefile): Add package initialization for Emacs batch
Enhance Makefile with package support by loading and initializing
MELPA archive before validating and compiling, ensuring required
packages are available during these operations.
feat(tests): Add unit tests for `wttrin--display-weather`
Introduce comprehensive tests for `wttrin--display-weather` function
to validate buffer creation, content, keybindings, and error
handling.
|
|
Session 1: Testing infrastructure and initial test coverage
Bug fixes in wttrin.el:
- Fix wttrin-additional-url-params to handle nil unit system
- Remove incorrect callback parameter to url-retrieve-synchronously
- Add nil buffer check for network failures
- Strip HTTP headers before decoding response
- Kill buffer after fetch to prevent memory leaks
- Fix double concatenation of URL params in cache function
- Add proper URL encoding via new wttrin--build-url function
Refactoring:
- Extract wttrin--build-url as pure, testable function
- Separate URL building logic from network I/O
Test infrastructure (33 tests, 100% passing):
- tests/testutil-wttrin.el: Shared test utilities
- tests/test-wttrin-additional-url-params.el: 7 tests
- tests/test-wttrin--make-cache-key.el: 9 tests
- tests/test-wttrin--build-url.el: 10 tests
- tests/test-wttrin--cleanup-cache-if-needed.el: 7 tests
Documentation:
- docs/testing-plan.org: Comprehensive testing roadmap
- docs/bugs.org: Bug analysis from code review
- docs/NOTES.org: Session tracking and guidelines
- docs/session-1-summary.org: Detailed session summary
Next session: Cache workflow tests, parsing logic extraction, integration tests
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
|
|
Implement a caching mechanism to reduce API calls and improve
performance. The cache stores weather data with configurable TTL (15
minutes default) and automatic cleanup when exceeding max entries
(50 default).
- Add cache configuration options for TTL and max entries
- Store fetched data with timestamps in hash table
- Add force refresh functionality with 'r' key binding
- Implement automatic cache cleanup to prevent unbounded growth
- Fall back to stale cache on network errors
- Also, fixed URL parameter concatenation in fetch function
|
|
|
|
- bind it locally instead within the let form
|
|
|
|
the unwieldly wttrin-default-accept-language became the lighter-weight and more
user-friendly wttrin-default-language
|
|
- fix let-binding for date-time-stamp and location-info
- also switched goto-line usage (interactive only) with forward-line
- require face-remap for buffer-face-mode-face
- adjust all code and docstring for 80 column width
|
|
|
|
|
|
- adjust display header to location and date
- change references from cities to location
- update default-location names
requery smoothly
- always name the buffer *wttr.in* for easy reuse/killing
- keep the buffer display up while user chooses next location
|
|
- added manual and straight installation instructions
- corrected font-height setting example
- corrected :type for wttrin-default-accept-language
- fix docstrings
corrected emacs-lisp code block, upcased usage header
|
|
- fixed types for default-accept-language and default-cities
- removed unneeded status variable in wttrin-fetch-raw-string
- syntax correction in defgroup wttrin description
- changed default font to Lucida Console (more universal + compatible with wttr.in)
- added games to keywords per package linter
- updated version, header description, and commentary
|
|
- changed license to GPLv3
- allow user to specify font name and height for display
- place date/time stamp above location name at top of buffer
- re-query with new location deletes to reduce buffer clutter
- updated screenshot to show current updates
- changed references from "city" to "location" in wttrin.el
- removed old CI config files
- removed old badges from readme file
|
|
Other changes being:
- added text prompt for quitting and selecting another city
- updated default cities
- fixed docstring issues reported by Emacs' linter
- updated several prompt texts
- updated maintainer
|
|
This fork has the following patches applied (still outstanding in bcbcarl/emacs-wttrin):
- patch for selecting the unit system: https://github.com/bcbcarl/emacs-wttrin/pull/10
- patch for url request to return ascii rather than html https://github.com/bcbcarl/emacs-wttrin/pull/18
- patch to use https instead of http: https://github.com/bcbcarl/emacs-wttrin/pull/15
|
|
|
|
If wttrin-default-cities only has the one city in it, default the input to
wttrin to that so that it's easier and faster to get the weather.
|
|
This means it can be easily called from other code, with the city passed as
an argument.
|
|
|
|
Local keymap allows to quit current buffer or query for new place.
|
|
|
|
* Depend on stable version of `xterm-color`.
* Use `erase-buffer` instead of delete all region.
* Replace `ido-completing-read` with `completing-read`.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
